Skip to content

Commit 42c343e

Browse files
committed
Address review
1 parent 9ba32a0 commit 42c343e

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

javascript/extractor/src/com/semmle/js/parser/JSONParser.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ private char next() throws ParseError {
7979
}
8080

8181
private char peek() {
82-
return offset < length ? src.charAt(offset) : (char) -1;
82+
return offset < length ? src.charAt(offset) : Character.MAX_VALUE;
8383
}
8484

8585
private JSONValue readValue() throws ParseError {
@@ -356,7 +356,7 @@ private void skipLineComment() throws ParseError {
356356
char c;
357357
next();
358358
next();
359-
while ((c = peek()) != '\r' && c != '\n') next();
359+
while ((c = peek()) != '\r' && c != '\n' && c != Character.MAX_VALUE) next();
360360
}
361361

362362
/** Skips the block comment starting at the current position. */
@@ -367,6 +367,7 @@ private void skipBlockComment() throws ParseError {
367367
next();
368368
do {
369369
c = peek();
370+
if (c == Character.MAX_VALUE) raise("Unterminated comment");
370371
next();
371372
if (c == '*' && peek() == '/') {
372373
next();

0 commit comments

Comments
 (0)