Closed
Bug 829758
Opened 12 years ago
Closed 12 years ago
Use MatchOnly mode for str_search().
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla21
People
(Reporter: sstangl, Unassigned)
References
Details
Attachments
(1 file)
|
(deleted),
patch
|
dvander
:
review+
|
Details | Diff | Splinter Review |
This doesn't affect any benchmark, but it's easy to do.
Attachment #701286 -
Flags: review?(dvander)
|
||
Updated•12 years ago
|
Attachment #701286 -
Flags: review?(dvander) → review+
|
Reporter | |
Comment 1•12 years ago
|
||
|
||
Comment 2•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
|
|
Reporter | |
Comment 3•12 years ago
|
||
I have backed out this patch pending investigation into Bug 832197. This patch does not appear to be wrong, but some of the underlying cross-compartment references get hairy, but apparently only in the case of str_search(). Can be re-landed once that mess is sorted out.
https://hg.mozilla.org/integration/mozilla-inbound/rev/2727e931b6c7
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
|
||
Comment 4•12 years ago
|
||
|
|
Reporter | |
Comment 5•12 years ago
|
||
Re-landed: Bug 832197 resolved in Bug 832217. Test cases included with this commit.
https://hg.mozilla.org/integration/mozilla-inbound/rev/8619a2942136
|
|
||
Comment 6•12 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 12 years ago → 12 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•