Commit 043f8c5

committed

fix #189, dynamic codegen broken on map serialization

1 parent c1db5fb commit 043f8c5Copy full SHA for 043f8c5

File tree

+10

-6

lines changed

src
- main/java/com/jsoniter/output
  - CodegenImplMap.java
- test/java/com/jsoniter/output
  - TestMap.java

+10

-6

lines changed

Lines changed: 0 additions & 5 deletions

Original file line number	Diff line number	Diff line change
`@@ -51,11 +51,6 @@ public static CodegenResult genMap(String cacheKey, ClassInfo classInfo) {`
`51`	`51`	`ctx.append("stream.writeMore();");`
`52`	`52`	`}`
`53`	`53`	`genWriteMapKey(ctx, keyType, noIndention);`
`54`		`- if (noIndention) {`
`55`		`- ctx.append("stream.write(':');");`
`56`		`- } else {`
`57`		`- ctx.append("stream.write((byte)':', (byte)' ');");`
`58`		`- }`
`59`	`54`	`if (isCollectionValueNullable) {`
`60`	`55`	`ctx.append("if (entry.getValue() == null) { stream.writeNull(); } else {");`
`61`	`56`	`CodegenImplNative.genWriteOp(ctx, "entry.getValue()", valueType, true);`

Lines changed: 10 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,5 @@`
`1`	`1`	`package com.jsoniter.output;`
`2`	`2`
`3`		`-import com.jsoniter.JsonIterator;`
`4`	`3`	`import com.jsoniter.spi.Config;`
`5`	`4`	`import com.jsoniter.spi.Encoder;`
`6`	`5`	`import com.jsoniter.spi.JsoniterSpi;`
`@@ -9,6 +8,7 @@`
`9`	`8`
`10`	`9`	`import java.io.ByteArrayOutputStream;`
`11`	`10`	`import java.io.IOException;`
	`11`	`+import java.math.BigDecimal;`
`12`	`12`	`import java.util.HashMap;`
`13`	`13`	`import java.util.Map;`
`14`	`14`
`@@ -147,4 +147,13 @@ public void test_object_key() {`
`147`	`147`	`m.put(1, 2);`
`148`	`148`	`assertEquals("{\"1\":2}", JsonStream.serialize(m));`
`149`	`149`	`}`
	`150`	`+`
	`151`	`+ public void test_multiple_keys() {`
	`152`	`+ HashMap<String, Object> map = new HashMap<String, Object>();`
	`153`	`+ map.put("destination", "test_destination_value");`
	`154`	`+ map.put("amount", new BigDecimal("0.0000101101"));`
	`155`	`+ map.put("password", "test_pass");`
	`156`	`+ final String serialized = JsonStream.serialize(map);`
	`157`	`+ assertEquals(-1, serialized.indexOf("::"));`
	`158`	`+ }`
`150`	`159`	`}`

Comments

(0)