`<regex>`: Implement collating ranges by muellerj2 · Pull Request #5238 · microsoft/STL · GitHub
Skip to content

<regex>: Implement collating ranges#5238

Merged
StephanTLavavej merged 18 commits into
microsoft:mainfrom
muellerj2:regex-implement-collating-ranges
Mar 24, 2025
Merged

<regex>: Implement collating ranges#5238
StephanTLavavej merged 18 commits into
microsoft:mainfrom
muellerj2:regex-implement-collating-ranges

Conversation

@muellerj2

Copy link
Copy Markdown

Resolves #5204. Follow-up to #5164 and #5209.

When compiled under /MD or /MDd, the new test cases specific to collating ranges are skipped:

Drive-by changes:

  • Mark _Parser::_Error() as [[noreturn]].
  • In _Matcher::_Skip() and _Matcher::_Do_class(), _STD qualify calls to some free functions and clean up the control flow that translates the current character and converts it to an unsigned value.

@muellerj2 muellerj2 requested a review from a team as a code owner January 15, 2025 18:28
Comment thread stl/inc/regex Outdated
Comment thread tests/std/tests/GH_005204_regex_collating_ranges/test.cpp
@CaseyCarter CaseyCarter added the bug Something isn't working label Jan 15, 2025
@StephanTLavavej StephanTLavavej added the regex meow is a substring of homeowner label Jan 15, 2025
@StephanTLavavej StephanTLavavej self-assigned this Jan 15, 2025
Comment thread stl/inc/regex Outdated
Comment thread tests/std/tests/GH_005204_regex_collating_ranges/test.cpp Outdated
Comment thread tests/std/tests/GH_005204_regex_collating_ranges/test.cpp Outdated
Comment thread tests/std/tests/GH_005204_regex_collating_ranges/test.cpp Outdated
Comment thread tests/std/tests/GH_005204_regex_collating_ranges/test.cpp Outdated
Comment thread tests/std/tests/GH_005204_regex_collating_ranges/test.cpp Outdated
Comment thread tests/std/tests/GH_005204_regex_collating_ranges/test.cpp
Comment thread tests/std/tests/GH_005204_regex_collating_ranges/test.cpp Outdated
Comment thread tests/std/tests/GH_005204_regex_collating_ranges/test.cpp
Comment thread tests/std/tests/GH_005204_regex_collating_ranges/test.cpp Outdated
@StephanTLavavej

Copy link
Copy Markdown
Member

@StephanTLavavej StephanTLavavej removed their assignment Mar 20, 2025
@StephanTLavavej StephanTLavavej moved this from Initial Review to Ready To Merge in STL Code Reviews Mar 20, 2025
@StephanTLavavej StephanTLavavej moved this from Ready To Merge to Merging in STL Code Reviews Mar 21, 2025
@StephanTLavavej StephanTLavavej self-assigned this Mar 21, 2025
@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 added a commit to StephanTLavavej/STL that referenced this pull request Mar 21, 2025
@StephanTLavavej StephanTLavavej merged commit 92cbb9b into microsoft:main Mar 24, 2025
@github-project-automation github-project-automation Bot moved this from Merging to Done in STL Code Reviews Mar 24, 2025
@StephanTLavavej

Copy link
Copy Markdown
Member

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

Labels

bug Something isn't working regex meow is a substring of homeowner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

<regex>: Collating ranges are broken

4 participants