Fix NaN bug and all broken tests

pquerna / ffjson

faster JSON serialization for Go

Apache License 2.0

2.97k stars 234 forks source link

Fix NaN bug and all broken tests #242

Closed erikdubbelboer closed 6 years ago

erikdubbelboer commented 6 years ago

I have combined all fixes I needed to do to make things testable again in the latest Go version. I thought this was better than making multiple pull requests. If you do want multiple separate ones let me know.

Fix Makefile to remove all generated files on clean
Return an error when trying to parse NaN (Fix #241)
Fix marshalling of RawMessage test (Closes #234)
Fix float errors in fuzz tests caused by default lib marshaling 0.00000012 as 1.2e-7 while ffjson doesn't
Remove invalid TODO

pquerna commented 6 years ago

Thank you for the PR and the misc fixes!

erikdubbelboer commented 6 years ago

Thank you for still maintaining this library.