Bug 222142 - [test262] test/built-ins/RegExp/match-indices needs updating for 'd' flag
Summary: [test262] test/built-ins/RegExp/match-indices needs updating for 'd' flag
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Michael Saboff
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-02-18 16:37 PST by Michael Saboff
Modified: 2021-02-25 15:46 PST (History)
2 users (show)

See Also:

Attachments
Patch (80.12 KB, patch)
2021-02-25 11:10 PST, Michael Saboff 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Saboff 2021-02-18 16:37:28 PST 
The current ToT test262 tests for the regexp-match-indices feature in the directory test/built-ins/RegExp/match-indices need updating for the recently approved 'd' flag.

This bug is to track when that gets done so we can enable the feature in JSTests/test262/config.yaml.
Comment 1 Michael Saboff 2021-02-18 20:57:03 PST 
There is a Pull request https://github.com/tc39/test262/pull/2934. with updated tests.  I downloaded that PR and found two bugs in the test, which I added comments for in the PR.
Comment 2 Michael Saboff 2021-02-25 11:10:12 PST 
Created attachment 421547 [details]
Patch
Comment 3 Yusuke Suzuki 2021-02-25 11:11:45 PST 
Comment on attachment 421547 [details]
Patch

r=me
Comment 4 EWS 2021-02-25 15:45:26 PST 
Committed r273518: <https://commits.webkit.org/r273518>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 421547 [details].
Comment 5 Radar WebKit Bug Importer 2021-02-25 15:46:15 PST 
<rdar://problem/74764286>