jshint / jshint / 2456

100%

Pull #2937

travis-ci

[[FIX]] Avoid crash when peeking past end of prog

The lexer returns `null` values when tokens are requested after all
input has been consumed. Previously, the `peek` function would correctly
produce the special "(end)" token in situations where the parser
attempted to look beyond the end of the program. In such cases, however,
it would also pollute the lookahead buffer with an invalid entry--the
`null` value returned by the lexer. Future calls to `peek` could receive
this buffered value.

Because JSHint's internals are written with the assumption that `peek`
always returns a token object, the `null` value would trigger a
TypeError and subsequent program crash.

Re-factor the `peek` function to only insert valid token objects into
the lookahead buffer.

5 of 5 new or added lines in 1 file covered. (100.0%)

3948 of 4042 relevant lines covered (97.67%)

6412.39 hits per line