Commit 301f44b

committed

fix #188 should not iterate when array is empty

1 parent 1ab2042 commit 301f44bCopy full SHA for 301f44b

File tree

-1

lines changed

src
- main/java/com/jsoniter/any
  - ArrayLazyAny.java
- test/java/com/jsoniter/any
  - TestList.java

-1

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -172,7 +172,7 @@ private Any fillCacheUntil(int target) {`
`172`	`172`	`if (lastParsedPos == head) {`
`173`	`173`	`if (!CodegenAccess.readArrayStart(iter)) {`
`174`	`174`	`lastParsedPos = tail;`
`175`		`- thrownewIndexOutOfBoundsException();`
	`175`	`+ returnnull;`
`176`	`176`	`}`
`177`	`177`	`Any element = iter.readAny();`
`178`	`178`	`cache.add(element);`

Lines changed: 8 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -1,10 +1,12 @@`
`1`	`1`	`package com.jsoniter.any;`
`2`	`2`
	`3`	`+import com.jsoniter.JsonIterator;`
`3`	`4`	`import junit.framework.TestCase;`
`4`	`5`
`5`	`6`	`import java.util.ArrayList;`
`6`	`7`	`import java.util.Arrays;`
`7`	`8`	`import java.util.Collections;`
	`9`	`+import java.util.Iterator;`
`8`	`10`
`9`	`11`	`public class TestList extends TestCase {`
`10`	`12`	`public void test_size() {`
`@@ -52,4 +54,10 @@ public void test_to_string() {`
`52`	`54`	`any.asList().add(Any.wrap(4));`
`53`	`55`	`assertEquals("[1,2,3,4]", any.toString());`
`54`	`56`	`}`
	`57`	`+`
	`58`	`+ public void test_for_each() {`
	`59`	`+ Any a = JsonIterator.deserialize("[]");`
	`60`	`+ Iterator<Any> iter = a.iterator();`
	`61`	`+ assertFalse(iter.hasNext());`
	`62`	`+ }`
`55`	`63`	`}`

Comments

(0)