Closed
Bug 1317308
Opened 8 years ago
Closed 8 years ago
\s should no longer match U+180E in regular expressions
Categories
(Core :: JavaScript: Standard Library, defect)
Core
JavaScript: Standard Library
Tracking
()
RESOLVED
FIXED
mozilla53
People
(Reporter: anba, Assigned: anba)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
13.43 KB,
patch
|
arai
:
review+
|
Details | Diff | Splinter Review |
`assertEq(/\s/.exec("\u180e"), null)` should pass.
Assignee | ||
Comment 1•8 years ago
|
||
We should probably generate the constants in irregexp/RegExpEngine.cpp through vm/make_unicode.py, but for now this quick fix should suffice.
Attachment #8810530 -
Flags: review?(arai.unmht)
Comment 2•8 years ago
|
||
Comment on attachment 8810530 [details] [diff] [review] bug1317308.patch Review of attachment 8810530 [details] [diff] [review]: ----------------------------------------------------------------- Great! yeah, I agree that the table should be automatically generated.
Attachment #8810530 -
Flags: review?(arai.unmht) → review+
Assignee | ||
Updated•8 years ago
|
Keywords: checkin-needed
Pushed by cbook@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/94615002eb6e Don't match Mongolian vowel separator as white space in regular expressions. r=arai
Keywords: checkin-needed
Comment 4•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/94615002eb6e
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox53:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Updated•7 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•