Skip to content

Conversation

@som-snytt
Copy link

The character class includes various chars.

The previous test was unlikely to fail, as it checked only
arbitrary chars, most of which are obviously not whitespace
even to the most broken implementation.

Note that it's not helpful to display an arbitrary
horizontal whitespace char after a failed test, though
I imagine it was a minimized input.

som-snytt added 2 commits May 17, 2025 18:37
The character class includes various chars.

The previous test was unlikely to fail, as it checked only
arbitrary chars, most of which are obviously not whitespace
even to the most broken implementation.

Note that it's not helpful to display an arbitrary
horizontal whitespace char after a failed test, though
I imagine it was a minimized input.
@som-snytt
Copy link
Author

@tpolecat turned the versions up to 11. Used -Wconf:stfu to ignore speed bumps.

Sorry I no longer know what my previous comment says, but it passes ++ test locally.

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.

1 participant