-
Notifications
You must be signed in to change notification settings - Fork 71
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Inconsistent test result of '+inf' #150
Comments
Is this causing practical issues for you? |
I just wrote a parser for TOML for fun and found that it cannot pass these test suites. |
I had to add custom processing to my test program to account for the inconsistent way the test suite encodes infinities https://github.com/glguy/toml-parser/blob/main/test-drivers/encoder/Main.hs#L59-L60 |
That's actually a different issue @glguy, with the "I" in "Inf" being capitalized sometimes. That was fixed last year. I'm not 100% sure if that's in the latest tagged release (need to look that up), but it's definitely in the latest master. Verified with:
(I made sure to rebuild the TomlDecoder/TomlEncoder with the changes). |
The
+inf
returnsin the
valid/spec/float-2.json
filetoml-test/tests/valid/spec/float-2.toml
Line 3 in 8bdabb2
toml-test/tests/valid/spec/float-2.json
Lines 6 to 9 in 8bdabb2
However, it returns
in the
valid/float/inf-and-nan.json
filetoml-test/tests/valid/float/inf-and-nan.toml
Line 8 in 8bdabb2
toml-test/tests/valid/float/inf-and-nan.json
Lines 10 to 13 in 8bdabb2
The text was updated successfully, but these errors were encountered: