Closed Bug 1338373 Opened 3 years ago Closed 3 years ago

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

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla54
Tracking Status
firefox54 --- fixed

People

(Reporter: arai, Assigned: arai)

References

Details

Attachments

(1 file)

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.