Skip to content

<regex>: Correctly reset capture groups for ECMAScript grammar #5456

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.

Sign up for GitHub

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

Merged
StephanTLavavej merged 13 commits into from
May 10, 2025
Merged

<regex>: Correctly reset capture groups for ECMAScript grammar #5456

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
cff04c6
rename matcher and add template parameter for future allocator support
muellerj3 May 1, 2025
5ef9267
adjust _Matcher2::_Ncap's type
muellerj3 May 1, 2025
3f6e6ec
remove `_Matcher2::_Get_ncap()`
muellerj3 May 1, 2025
f252cab
remove duplicate member `_Matcher2::_First` in favor of `_Matcher2::_…
muellerj3 May 1, 2025
1d225e8
`_Do_rep`: Remove unnecessary local variable `_Cur_iter` and replace …
muellerj3 May 1, 2025
90f84c2
Add capture group range to stored loop state and properly reset captu…
muellerj3 May 1, 2025
7abea50
remove nullptr
muellerj3 May 1, 2025
31b46db
extend test coverage
muellerj3 May 5, 2025
76bea8c
Merge branch 'main' into regex-fix-capture-group-reset
StephanTLavavej May 8, 2025
cf42b7f
Add DMIs to `_Loop_vals_v2_t`.
StephanTLavavej May 8, 2025
9d5a528
Extract `_Any_posix`.
StephanTLavavej May 8, 2025
f5b3ebb
Scope `_Node_if* _Node` to the for-loop.
StephanTLavavej May 8, 2025
2aa8ccf
Avoid unnecessary braces in switches.
StephanTLavavej May 8, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'main' into regex-fix-capture-group-reset
  • Loading branch information
@StephanTLavavej
StephanTLavavej committed May 8, 2025
commit 76bea8cf1822a59ae29f8109078ed45b763a8bd1

This merge commit was added into this branch cleanly.

There are no new changes to show, but you can still view the diff.