search

chadaustin / sajson

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

Fix double precision conformance issues exposed by https://github.com/miloyip/nativejson-benchmark #10

Open chadaustin opened 8 years ago

chadaustin commented 8 years ago

See https://rawgit.com/miloyip/nativejson-benchmark/master/sample/conformance.html#1.%20Parse%20Validation

aggsol commented 7 years ago

Is this still on your agenda?

chadaustin commented 7 years ago

Hi @aggsol -- the only conformance issues are around the precision of very large and very small doubles, and when I looked into this last it looked like the only precision issues were at the least-significant digits. That said, I do think this is worth improving. I haven't prioritized it because many JSON encoders have precision issues for large and tiny doubles and I've yet to need it. If you'd like, it might be fun to write up a test case and see why precision is getting lost.