parseJSON validation is inaccurate

[anonymous]

The current method for validating JSON for parsing is inaccurate and buggy. For example, jQuery.parseJSON("{,}") passes the existing test, and results in a syntax error when trying to eval it, where jquery should handle the error.

This patch introduces an alternative validation method that really ensures that the input is valid JSON before evaluating it:

 https://github.com/jquery/jquery/pull/516

JQuery 1.4.1+ affected on all platforms

[rwaldron]

That error is exactly correct, "{,}" is not valid

 http://gyazo.com/88815ad637cc0fccc9ee58751c45d0f9.png  http://gyazo.com/fe06c62923b84b8b5a5aa8939fd43fb9.png

jsonlint...  http://gyazo.com/a604c8ccff158be76b83288b337d2a0c.png

[anonymous]

The behavior is inconsistent, though, because the method of validation is bogus.

 http://i.imgur.com/xf9NW.png

[rwaldron]

Sorry, but that "json" string is the only bogus thing here...

 http://gyazo.com/5c435cb67293172ba715768f4389d4de.png

[anonymous]

But why is it okay that {"a":5,} produces a different exception than {'a':5,}? It means that code handling the resulting exceptions has to handle two cases for the same error.

[rwaldron]

They both produce an "unexpected token" syntax error when using native JSON.parse()

Why is it so hard to create _valid_ JSON?