Do not call _yamlParser.hasNext() to wrap the exception

asomov · asomov · commit 2cfc7d7331af · 2018-05-11T23:58:49.000+02:00
diff --git a/yaml/src/main/java/com/fasterxml/jackson/dataformat/yaml/YAMLParser.java b/yaml/src/main/java/com/fasterxml/jackson/dataformat/yaml/YAMLParser.java
@@ -282,7 +282,7 @@ public JsonToken nextToken() throws IOException
             return null;
         }
 
-        while (_yamlParser.hasNext()) {
+        while (true /*_yamlParser.hasNext()*/) {
             Event evt;
             try {
                 evt = _yamlParser.next();
@@ -386,8 +386,6 @@ public JsonToken nextToken() throws IOException
                     continue;
             }
         }
-        //TODO what should be thrown here ?
-        throw new RuntimeException("Unexpected events.");
     }
 
     protected JsonToken _decodeScalar(ScalarEvent scalar) throws IOException

Original file line number	Diff line number	Diff line change
`@@ -282,7 +282,7 @@ public JsonToken nextToken() throws IOException`
`282`	`282`	`return null;`
`283`	`283`	`}`
`284`	`284`
`285`		`- while (_yamlParser.hasNext()) {`
	`285`	`+ while (true /_yamlParser.hasNext()/) {`
`286`	`286`	`Event evt;`
`287`	`287`	`try {`
`288`	`288`	`evt = _yamlParser.next();`
`@@ -386,8 +386,6 @@ public JsonToken nextToken() throws IOException`
`386`	`386`	`continue;`
`387`	`387`	`}`
`388`	`388`	`}`
`389`		`- //TODO what should be thrown here ?`
`390`		`- throw new RuntimeException("Unexpected events.");`
`391`	`389`	`}`
`392`	`390`
`393`	`391`	`protected JsonToken _decodeScalar(ScalarEvent scalar) throws IOException`