Closed
Bug 1138095
Opened 10 years ago
Closed 10 years ago
Fix in-tree consumers that use non-standard flag argument of String.prototype.{search,match,replace} in mail/.
Categories
(Thunderbird :: General, defect)
Thunderbird
General
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 39.0
People
(Reporter: arai, Assigned: arai)
References
Details
Attachments
(1 file)
|
3.66 KB,
patch
|
mkmelin
:
review+
|
Details | Diff | Splinter Review |
(same as bug 1131107)
Before fixing bug 1108382, we need to replace all of them with regexp literal, new RegExp(...) or .split().join().
| Assignee | ||
Comment 1•10 years ago
|
||
Fixed String.prototype.replace call sites to use regexp literals.
at least no regression on try run: https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=7a729c78766a
Attachment #8570975 -
Flags: review?(mkmelin+mozilla)
Comment 2•10 years ago
|
||
Comment on attachment 8570975 [details] [diff] [review]
Do not use non-standard flag argument of String.prototype.replace in mail/.
Review of attachment 8570975 [details] [diff] [review]:
-----------------------------------------------------------------
LGTM, thx! r=mkmelin
Attachment #8570975 -
Flags: review?(mkmelin+mozilla) → review+
| Assignee | ||
Comment 3•10 years ago
|
||
Thank you! :D
https://hg.mozilla.org/comm-central/rev/da5fd11104e5
| Assignee | ||
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 39.0
| Assignee | ||
Updated•10 years ago
|
Assignee: nobody → arai.unmht
You need to log in
before you can comment on or make changes to this bug.
Description
•