Summary: | ES6 Change: Unify handling of RegExp CharacterClassEscapes \w and \W and Word Asserts \b and \B | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Michael Saboff <msaboff> | ||||
Component: | JavaScriptCore | Assignee: | Michael Saboff <msaboff> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, keith_miller, mark.lam, mathias, saam, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | Other | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Michael Saboff
2016-06-07 17:37:39 PDT
Created attachment 280755 [details]
Patch
Comment on attachment 280755 [details]
Patch
r=me
Will this cause some apparent failures in the kangax or ECMA test suites? If so, I think we should hold off until the tests are updated.
(In reply to comment #3) > Comment on attachment 280755 [details] > Patch > > r=me > > Will this cause some apparent failures in the kangax or ECMA test suites? If > so, I think we should hold off until the tests are updated. It does cause a kangax failure. I fully expect that is also causes a test262 failure. I'll hold off landing until the spec is updated as well as the tests. *** Bug 159055 has been marked as a duplicate of this bug. *** https://github.com/tc39/ecma262/pull/525 was merged. New expectations: /\w/iu.test('\u017F') // true /\w/iu.test('\u212A') // true /\W/iu.test('\u017F') // false /\W/iu.test('\u212A') // false /\W/iu.test('s') // false /\W/iu.test('S') // false /\W/iu.test('K') // false /\W/iu.test('k') // false Committed r202490: <http://trac.webkit.org/changeset/202490> *** Bug 160523 has been marked as a duplicate of this bug. *** *** Bug 160523 has been marked as a duplicate of this bug. *** |