Summary: | RegExpBuiltinExec should create "groups" property unconditionally | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Alexey Shvayka <ashvayka> | ||||
Component: | JavaScriptCore | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Minor | CC: | commit-queue, ews-watchlist, keith_miller, mark.lam, msaboff, ross.kirsling, saam, tzagallo, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 202475, 204264 | ||||||
Attachments: |
|
Description
Alexey Shvayka
2019-11-11 06:03:44 PST
Created attachment 383266 [details]
Patch
I am adding this bug as a blocker of RegExp Match Indices proposal (https://bugs.webkit.org/show_bug.cgi?id=202475), because "indices" property should be created after "groups" (order is observable), and the way we currently add properties in RegExpMatchesArray.cpp/createStructureImpl will make that unnecessarily tricky. Comment on attachment 383266 [details]
Patch
Nice!
Comment on attachment 383266 [details] Patch Clearing flags on attachment: 383266 Committed r252374: <https://trac.webkit.org/changeset/252374> All reviewed patches have been landed. Closing bug. |