search

chadaustin / sajson

Lightweight, extremely high-performance JSON parser for C++11
MIT License
565 stars 41 forks source link

Conformance report for reference #14

Closed miloyip closed 8 years ago

miloyip commented 8 years ago

https://github.com/miloyip/nativejson-benchmark/blob/master/sample/conformance_sajson%20(C%2B%2B).md

chadaustin commented 8 years ago

Thanks, @miloyip, for the bug reports! Your validation suite is excellent.

I fixed some of the issues:

https://github.com/chadaustin/sajson/commit/a1a4b8db9fb363fbf0c7ef12a7d2d0c0fc1ee448 https://github.com/chadaustin/sajson/commit/9992758580311723162f9dfbdeb19373e1e95f62 https://github.com/chadaustin/sajson/commit/4272e4d1b67f8f8b40acb5bcc8524f1f4470422a https://github.com/chadaustin/sajson/commit/9c79e8874cb9efbd15b8866adc6bcb1cd9f1648d

The double precision edge cases are a won't-fix because the JSON RFC says implementations are allowed to have unspecified precision, and sajson was built to quickly parse "normalized" numeric values.

Thanks again!

chadaustin commented 8 years ago

Oh, and p.s., I tried to build the json validation suite on my Ubuntu 16.04 laptop but getting premake5 was proving to be challenging so I gave up. If you see that the latest sajson regresses in any way, please let me know!