Skip to content
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

Issue276 plus/minus handling in parse_infnan #277

Merged
merged 5 commits into from
Nov 19, 2024

Conversation

dalle
Copy link
Collaborator

@dalle dalle commented Nov 17, 2024

"-+NNNN" should fail to parse, regardless of what NNNN is and regardless if FASTFLOAT_ALLOWS_LEADING_PLUS is defined.

Fixes #276

@lemire
Copy link
Member

lemire commented Nov 18, 2024

Running tests. Some of the CI failures are not related (caused by) this PR, and can be handled separately.

@dalle
Copy link
Collaborator Author

dalle commented Nov 19, 2024

I missed to do the "one thing" that the pull request template said I should to, formatting the code according to clang-format. 😅

@lemire
Copy link
Member

lemire commented Nov 19, 2024

missed to do the "one thing" that the pull request template said I should to, formatting the code according to clang-format.

Yeah but that's not a blocking issue because we can always do it later.

@lemire
Copy link
Member

lemire commented Nov 19, 2024

Running tests.

@lemire
Copy link
Member

lemire commented Nov 19, 2024

I am merging this. The failing tests are unrelated and will be fixed later.

@lemire lemire merged commit 8e5f76c into fastfloat:main Nov 19, 2024
37 of 39 checks passed
@dalle dalle deleted the issue276-plus-minus-infnan branch November 19, 2024 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Parsing -+nan[(n-char-seq-opt)] and -+inf[inity] succeeds with FASTFLOAT_ALLOWS_LEADING_PLUS
2 participants