Match updated spec for `/\b/iu` and `/\B/iu`

RESOLVED FIXED in Firefox 54

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: arai, Assigned: arai)

Tracking

Trunk
mozilla54
Points:
---

Firefox Tracking Flags

(firefox54 fixed)

Details

Attachments

(1 attachment)

related to bug 1281739.
\b and \B should also be updated to follow the change in https://github.com/tc39/ecma262/pull/525
* Changed BoyerMoorePositionInfo to use kIgnoreCaseWordRanges for iu flags
  * Changed EmitWordCheck to avoid using optimized path for iu flags
    (maybe we could add optimized path tho...)
  * Changed EmitWordCheck to check 0x017F and 0x212A for iu flags
Assignee: nobody → arai.unmht
Status: NEW → ASSIGNED
Attachment #8835803 - Flags: review?(till)
Comment on attachment 8835803 [details] [diff] [review]
Update word boundary in RegExp with unicode and ignoreCase flags to include U+017F and U+212A.

Review of attachment 8835803 [details] [diff] [review]:
-----------------------------------------------------------------

Looks great, thanks!
Attachment #8835803 - Flags: review?(till) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/c043f1737e222180549cb754ddd2e83f0d2223bd
Bug 1338373 - Update word boundary in RegExp with unicode and ignoreCase flags to include U+017F and U+212A. r=till
Duplicate of this bug: 1321664
https://hg.mozilla.org/mozilla-central/rev/c043f1737e22
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in before you can comment on or make changes to this bug.