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

fix the error message in case the quantifier maximum (second value) is smaller than the minimum (first value) #1260

Merged
merged 2 commits into from
Oct 8, 2022

Conversation

rbri
Copy link
Collaborator

@rbri rbri commented Sep 16, 2022

This supersedes #1245

@rbri
Copy link
Collaborator Author

rbri commented Sep 16, 2022

Have tested this with

real Chrome - Error: Invalid regular expression: /b{2,1}/: numbers out of order in {} quantifier

real FF - Error: numbers out of order in {} quantifier.

Hope my version of the error msg is at least as good a the two.

They french version was created using DeepL.

@rbri rbri merged commit 3e5489a into mozilla:master Oct 8, 2022
@rbri rbri mentioned this pull request Oct 8, 2022
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.

None yet

1 participant