Merge pull request #363 from orisano/fix/#359

fix: to care about the case of OpInterfacePtr
goccy · Apr 21, 2022 · 337d02f · 337d02f
2 parents 171d975 + 6db1acf
commit 337d02f
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/decode_test.go b/decode_test.go
@@ -3918,3 +3918,16 @@ func TestIssue360(t *testing.T) {
 		t.Errorf("unexpected result: %v", uints)
 	}
 }
+
+func TestIssue359(t *testing.T) {
+	var a interface{} = 1
+	var b interface{} = &a
+	var c interface{} = &b
+	v, err := json.Marshal(c)
+	if err != nil {
+		t.Errorf("unexpected error: %v", err)
+	}
+	if string(v) != "1" {
+		t.Errorf("unexpected result: %v", string(v))
+	}
+}
diff --git a/internal/encoder/opcode.go b/internal/encoder/opcode.go
@@ -363,7 +363,7 @@ func copyOpcode(code *Opcode) *Opcode {
 
 func setTotalLengthToInterfaceOp(code *Opcode) {
 	for c := code; !c.IsEnd(); {
-		if c.Op == OpInterface {
+		if c.Op == OpInterface || c.Op == OpInterfacePtr {
 			c.Length = uint32(code.TotalLength())
 		}
 		c = c.IterNext()