search

tidwall / gjson

Get JSON values quickly - JSON parser for Go
MIT License
14.12k stars 847 forks source link

Incorrect edge cases for gjson.Valid() #210

Closed WillAbides closed 3 years ago

WillAbides commented 3 years ago

I ran test_parsing from https://github.com/nst/JSONTestSuite against gjson.Valid() and discovered a couple of edge cases that gjson incorrectly calls valid.

[-] from n_array_just_minus.json

and

[-.123] from n_number_neg_real_without_int_part.json

To be clear, this isn't a problem for me. I'm just reporting it because I thought you might want to know.

tidwall commented 3 years ago

To be clear, this isn't a problem for me. I'm just reporting it because I thought you might want to know.

That's super kind of you. Thank you for reporting! :)