Skip to content

<regex>: Fix goofy message for error_badbrace - #5025

Merged
Stephan T. Lavavej (StephanTLavavej) merged 1 commit into
microsoft:mainfrom
5AIPAVAN:branch-3
Oct 21, 2024
Merged

<regex>: Fix goofy message for error_badbrace#5025
Stephan T. Lavavej (StephanTLavavej) merged 1 commit into
microsoft:mainfrom
5AIPAVAN:branch-3

Conversation

@5AIPAVAN

Copy link
Copy Markdown
Contributor

Fixes #4993
updated line 502 in regex.cpp
from : "contained an invalid range in a { expression }."
to : "contained an invalid range in a {} expression."

@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) added the enhancement Something can be improved label Oct 17, 2024
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) changed the title updated line 502 in regex.cpp to solve issue <regex>: Goofy message for error_badbrace #4993 <regex>: Fix goofy message for error_badbrace Oct 17, 2024
@StephanTLavavej

Copy link
Copy Markdown
Member

I've updated the PR title to fix several issues:

  • The <regex> header is being edited here, not a file named regex.cpp
  • PR titles shouldn't mention hyper-details like line numbers
  • PR titles don't need to mention issue numbers; these are meaningless when viewing commit history

@StephanTLavavej

Copy link
Copy Markdown
Member

I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed.

@StephanTLavavej
Stephan T. Lavavej (StephanTLavavej) merged commit 27b1f88 into microsoft:main Oct 21, 2024
@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks for this diagnostic improvement and congrats on your second microsoft/STL commit! 🎉 🎉 😺

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Something can be improved regex meow is a substring of homeowner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

<regex>: Goofy message for error_badbrace

2 participants