fix:(conv/j2t) not recurse in tb_write_default_or_empty to avoid dead-looping on cycle-referred struct (#89)

* fix: judge escape string wrong

* fix: not recurse in `tb_write_default_or_empty` to avoid dead-loop

* change test

* update README

Author: AsterDY

Makefile

@@ -31,7 +31,7 @@ CFLAGS_avx		:= -msse -mno-sse4 -mavx -mpclmul -mno-avx2 -mstack-alignment=0 -DUS
 CFLAGS_avx2		:= -msse -mno-sse4 -mavx -mpclmul -mavx2 -mstack-alignment=0 -DUSE_AVX=1 -DUSE_AVX2=1 
 CFLAGS_sse		:= -msse -mno-sse4 -mno-avx -mno-avx2 -mpclmul
 
-CC_amd64		:= /opt/homebrew/Cellar/llvm\@14/14.0.6/bin/clang
+CC_amd64		:= clang
 ASM2ASM_amd64	:= tools/asm2asm/asm2asm.py
 
 CFLAGS			:= -mno-red-zone

README.md

@@ -12,11 +12,9 @@ Dynamic-Go for Protobuf protocol: [introduction.md](./proto/INTRODUCTION.md)
 
 ### thrift
 Thrift IDL parser and message operators. It can parse thrift IDL in runtime and handle thrift data in generic way.
-[DOC](thrift/README.md)
 
 #### thrift/generic
 Reflection APIs to search, modify, deserialize, serialize thrift value **with or without** runtime type descriptor.
-[DOC](thrift/generic/README.md)
 
 #### thrift/base 
 The meta data about message transportation, including caller, address, log-id, etc. It is mainly used for `conv` (protocol convertion) modules.
@@ -26,40 +24,32 @@ Built-in implementation of thrid-party annotations, see [thrift_idl_annotation_s
 
 ### proto
 Protobuf IDL parser and message operators. It can parse protobuf IDL in runtime and handle protobuf data in generic way.
-[DOC](proto/README.md)
 
 #### proto/generic
 Reflection APIs to search, modify, deserialize, serialize protobuf value **with or without** runtime descriptor.
-[DOC](proto/generic/README.md)
 
 #### proto/protowire
 Protobuf data encode and decode APIs. It parses and formats the low-level raw wire encoding. It is modified from Protobuf official code [`encoding/protowire`](https://pkg.go.dev/google.golang.org/protobuf/encoding/protowire).
-[DOC](proto/protowire/README.md)
 
 #### proto/binary
 BinaryProtocol tool for Protobuf Protocol. It can read, wirte and skip fields directly on binary data of protobuf message.
-[DOC](proto/binary/README.md)
 
 ### http
 Http request/response wrapper interfaces. They are mainly used to pass http values on `http<>thrift` conversion. 
-[DOC](http/README.md)
 
 ### conv
 Protocol convertors. Based on reflecting ability of `thrift`, `json` and `protobuf` modules, it can convert message from one protocol into another. 
 
 #### conv/j2t
 Convert JSON value or JSON-body HTTP request into thrift message.
-[DOC](conv/j2t/README.md)
 
 #### conv/t2j
 Convert thrift message to JSON value or JSON-body HTTP response.
-[DOC](conv/t2j/README.md)
 
 #### conv/j2p
 Convert JSON value into protobuf message.
-[DOC](conv/j2p/README.md)
 
 #### conv/p2j
 Convert protobuf message into JSON value.
-[DOC](conv/p2j/README.md)
+
 

conv/j2t/conv_amd64_test.go

@@ -41,10 +41,6 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
-func init() {
-	recursivelyWrite = true
-}
-
 func TestError(t *testing.T) {
 	desc := getExampleDesc()
 

conv/j2t/conv_test.go

@@ -349,17 +349,11 @@ func getNullErrData() []byte {
 	return out
 }
 
-var recursivelyWrite = false
-
 func TestWriteDefault(t *testing.T) {
 	desc := getExampleDesc()
 	data := []byte(`{"Path":"<>"}`)
 	exp := example3.NewExampleReq()
 	exp.InnerBase = &example3.InnerBase{}
-	println(recursivelyWrite)
-	if recursivelyWrite {
-		exp.InnerBase = sample.GetEmptyInnerBase3()
-	}
 	data2 := []byte(`{"Path":"<>","Base":{}}`)
 	err := json.Unmarshal(data2, exp)
 	require.Nil(t, err)
@@ -400,9 +394,6 @@ func TestWriteRequired(t *testing.T) {
 	t.Run("http-mapping", func(t *testing.T) {
 		exp := example3.NewExampleReq()
 		exp.InnerBase = &example3.InnerBase{}
-		if recursivelyWrite {
-			exp.InnerBase = sample.GetEmptyInnerBase3()
-		}
 		data2 := []byte(`{"Path":"","Base":{}}`)
 		err := json.Unmarshal(data2, exp)
 		require.Nil(t, err)

conv/j2t/impl_fallback.go

@@ -190,7 +190,7 @@ func (self *BinaryConv) doRecurse(ctx context.Context, s string, jp int, desc *t
 					ret = jp
 
 					var key string
-					if v.Ep >= int64(ret) {
+					if v.Ep >= 0 && v.Ep < int64(ret) {
 						key, err = strconv.Unquote(s[v.Iv-1 : ret])
 						if err != nil {
 							return
@@ -271,7 +271,7 @@ func (self *BinaryConv) doRecurse(ctx context.Context, s string, jp int, desc *t
 					ret = jp
 
 					var key string
-					if v.Ep >= int64(ret) {
+					if v.Ep >= 0 && v.Ep < int64(ret) {
 						key, err = strconv.Unquote(s[v.Iv-1 : ret])
 						if err != nil {
 							return