Closed Bug 830783 Opened 7 years ago Closed 7 years ago

RegExp rooting

Categories

(Core :: JavaScript Engine, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla21

People

(Reporter: Ms2ger, Assigned: Ms2ger)

References

Details

Attachments

(3 files)

No description provided.
Attached patch Part a: DoMatchSplinter Review
Attachment #702321 - Flags: review?(terrence)
Attachment #702325 - Flags: review?(terrence)
Comment on attachment 702321 [details] [diff] [review]
Part a: DoMatch

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

Nice.
Attachment #702321 - Flags: review?(terrence) → review+
Comment on attachment 702324 [details] [diff] [review]
Part b: CreateRegExpMatchResult and ExecuteRegExpLegacy

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

Very nice.
Attachment #702324 - Flags: review?(terrence) → review+
Comment on attachment 702325 [details] [diff] [review]
Part c: random stuff

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

\o/
Attachment #702325 - Flags: review?(terrence) → review+
https://hg.mozilla.org/mozilla-central/rev/9990c861edc8
https://hg.mozilla.org/mozilla-central/rev/b2202290801d
https://hg.mozilla.org/mozilla-central/rev/8ba0f95510eb
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
You need to log in before you can comment on or make changes to this bug.