ThisingL commented on code in PR #2638:
URL: https://github.com/apache/fory/pull/2638#discussion_r2366280993


##########
go/fory/tests/structs.go:
##########
@@ -18,11 +18,38 @@
 package fory
 
 // ValidationDemo is a simple struct for testing code generation
-// Contains only basic types since PR1 only supports basic types
+// Contains various basic types to validate comprehensive type support
 
 // fory:gen
 type ValidationDemo struct {
-       A int32  `json:"a"` // int32 field
-       B string `json:"b"` // string field
-       C int64  `json:"c"` // int64 field (instead of array, as arrays are not 
supported yet)
+       A int32   // int32 field
+       B string  // string field
+       C int64   // int64 field
+       D float64 // float64 field
+       E bool    // bool field
+}
+
+// SliceDemo is a struct for testing slice serialization
+// Contains various slice types
+
+// fory:gen
+type SliceDemo struct {
+       IntSlice    []int32   // slice of int32
+       StringSlice []string  // slice of string
+       FloatSlice  []float64 // slice of float64
+       BoolSlice   []bool    // slice of bool
+}
+
+// DynamicSliceDemo is a struct for testing dynamic slice serialization
+// fory:gen
+type DynamicSliceDemo struct {
+       DynamicSlice []interface{} // slice of interface{}
+}
+
+// MapDemo demonstrates map field support
+// fory:gen

Review Comment:
   yeah, I will update this in a new PR



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to