Skip to content

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
# for free to join this conversation on GitHub. Already have an account? # 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