Skip to content

Conversation

khwilliamson
Copy link
Contributor

Prior to this commit, it was claimed that certain sequences should not occur in regular expression pattern bracketed character classes. I tried each one changed here, and they worked fine.

  • This set of changes does not require a perldelta entry.

@book
Copy link
Contributor

book commented Sep 6, 2025

Does this mean that the regular expression [\LF]O\Qo will match the string foo?

@book
Copy link
Contributor

book commented Sep 6, 2025

It does. Wow.

Prior to this commit, it was claimed that certain sequences should not
occur in regular expression pattern bracketed character classes.  I
tried each one changed here, and they worked fine.
@khwilliamson
Copy link
Contributor Author

@book, I did some more experiments, and changed the text for \Q and \E accordingly since your approval

@jkeenan
Copy link
Contributor

jkeenan commented Sep 6, 2025

@book, I did some more experiments, and changed the text for \Q and \E accordingly since your approval

Do we already have unit tests in our test suite which verify the behavior which you are now documenting? If not, we should write such tests.

@book
Copy link
Contributor

book commented Sep 6, 2025

The experiments should be turned into tests, basically.

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.

3 participants