tree 269d64ab6de008c6ad971563dacc94149a047333
parent c3a986600f9975a33e3573d85b48db63011d3711
author vslashg <veloso@verylowsodium.com> 1725928216 -0400
committer GitHub <noreply@github.com> 1725928216 -0700
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsFcBAABCAAQBQJm35MZCRC1aQ7uu5UhlAAAAFoQABc55fgiso+LBcBbgZUnzYH2
 pErX6f3UrPz4H0DftRHSPUQE7UHxUxj6xeV8FASwwdhGxYvXmYbyPSEFoQHYlOpt
 6ZAqZb0TuWztD0c++Xf1aUJlDDzoUGFoujgcRBvNizFYRT5WFOq90T6nBH47hqgC
 RtacMBCjynN/GFX+ovyNVSC7BPpjxpnYMv6+1GY4GD3DDvBJNh881pKLpnH9jpa2
 4CdWl9KgDkLz8pf44NpOPlmwFI2O2uH0wuOAfMHG0kDaA8ucnBlKXio6SMsbzY9A
 xXj80m2hZiFdISw4ZP4CsTdy7je2yG0cFB4PyRvKwPbp+jEoZ9eCph+4TDsSgAg4
 iaLdiuChDoDft5WeIVBB1fqsPiDlr7qFy3Am9t1oY89/aNwLvrXx5liiT+2FrwIp
 AxneZftIh96kgS3i0qj7fY332jITluYkAAv1mS7bp+n2xzBROGadn2rgTm1ODSNU
 mWa5m9ecn/Fcj3HvNP7M24yUgGC8ynw46eAba9B/fx+q/YP9bjaTuLFpd6gzZOkW
 JdMf6rPWULovJfF+oirebW/hANkHu41esctEJhYIceXji6pJtEsoKo+LYAuJARU7
 Y5hLvCghFnbsis9MKfscok1B9HS9ti7Js0y6IfSJDA+Y69n6caABQTanVrPTAdyL
 XAOYHvE0mE3vOXm3QaQ5
 =MUxN
 -----END PGP SIGNATURE-----
 

Fix a parser bug where tokens are misidentified as commas. (#1502)

* Fix a parser bug where tokens are misidentified as commas.

In the old and new readers, when parsing an object, a comment
followed by any non-`}` token is treated as a comma.

The new unit test required changing the runjsontests.py
flag regime so that failure tests could be run with default settings.

* Honor allowComments==false mode.

Much of the comment handling in the parsers is bespoke, and does not
honor this flag.  By unfiying it under a common API, the parser is
simplified and strict mode is now more correctly strict.

Note that allowComments mode does not allow for comments in
arbitrary locations; they are allowed only in certain positions.
Rectifying this is a bigger effort, since collectComments mode requires
storing the comments somewhere, and it's not immediately clear
where in the DOM all such comments should live.

---------

Co-authored-by: Jordan Bayles <bayles.jordan@gmail.com>