1281739 - Match updated spec for `/\w/iu` and `/\W/iu`

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Mathias Bynens]

Now that https://github.com/tc39/ecma262/pull/525 has landed, `\u017F` (LATIN SMALL LETTER LONG S) and `\u212A` (KELVIN SIGN) are word characters for `/iu` patterns.

Expected behavior:

    /\w/iu.test('\u017F') // true
    /\w/iu.test('\u212A') // true
    /\W/iu.test('\u017F') // false
    /\W/iu.test('\u212A') // false

Actual behavior: currently, they all result in `true`.

Comment 1

[Tooru Fujisawa [:arai]]

Attachment: Do not match K, S, k, s, KELVIN SIGN, LATIN SMALL LETTER LONG S with \W in unicode RegExp.

Removed K, S, k, s, KELVIN SIGN, LATIN SMALL LETTER LONG S from kNegatedIgnoreCaseWordAndSurrogateRanges.

Comment 2

[Till Schneidereit [:till]]

Comment on attachment 8764761 [details] [diff] [review]
Do not match K, S, k, s, KELVIN SIGN, LATIN SMALL LETTER LONG S with \W in unicode RegExp.

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

Looks good, thank you.

Comment 3

[Tooru Fujisawa [:arai]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/71d499e82b2cd5096b6f92f0c565cb6cfd5c6039
Bug 1281739 - Do not match K, S, k, s, KELVIN SIGN, LATIN SMALL LETTER LONG S with \W in unicode ignoreCase RegExp. r=till

Comment 4

[Tooru Fujisawa [:arai]]

This is a breaking change.
will update documentation after merge.

Comment 5

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/71d499e82b2c

Comment 6

[Kohei Yoshino]

I'm not going to write a site compatibility doc for this because the u flag is still new and the change is minor.

Comment 7

[Tooru Fujisawa [:arai]]

https://developer.mozilla.org/en-US/Firefox/Releases/50