Closed
Bug 803907
Opened 12 years ago
Closed 12 years ago
Miscellaneous GC cleanups
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla19
People
(Reporter: billm, Assigned: billm)
Details
Attachments
(4 files)
4.25 KB,
patch
|
terrence
:
review+
|
Details | Diff | Splinter Review |
6.25 KB,
patch
|
terrence
:
review+
|
Details | Diff | Splinter Review |
1.94 KB,
patch
|
terrence
:
review+
|
Details | Diff | Splinter Review |
2.69 KB,
patch
|
terrence
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•12 years ago
|
||
This appears to be unused.
Attachment #673626 -
Flags: review?(terrence)
Assignee | ||
Comment 2•12 years ago
|
||
I have never used this code before. It just seems like a hindrance.
Attachment #673627 -
Flags: review?(terrence)
Assignee | ||
Comment 4•12 years ago
|
||
This must have been refactored from one of the old hashtable implementations.
Attachment #673629 -
Flags: review?(terrence)
Comment 5•12 years ago
|
||
Comment on attachment 673626 [details] [diff] [review] remove MapRoots Review of attachment 673626 [details] [diff] [review]: ----------------------------------------------------------------- Thanks, I've also been meaning to kill this!
Attachment #673626 -
Flags: review?(terrence) → review+
Comment 6•12 years ago
|
||
Comment on attachment 673627 [details] [diff] [review] remove some debugging code Review of attachment 673627 [details] [diff] [review]: ----------------------------------------------------------------- I haven't used this either.
Attachment #673627 -
Flags: review?(terrence) → review+
Comment 7•12 years ago
|
||
Comment on attachment 673628 [details] [diff] [review] remove unused functions Review of attachment 673628 [details] [diff] [review]: ----------------------------------------------------------------- Wfm.
Attachment #673628 -
Flags: review?(terrence) → review+
Comment 8•12 years ago
|
||
Comment on attachment 673629 [details] [diff] [review] simplify hashtable iteration Review of attachment 673629 [details] [diff] [review]: ----------------------------------------------------------------- Yup. I've been meaning to do this one too.
Attachment #673629 -
Flags: review?(terrence) → review+
Assignee | ||
Comment 9•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/9679ec5b0b33 https://hg.mozilla.org/integration/mozilla-inbound/rev/fb81587d2dd8 https://hg.mozilla.org/integration/mozilla-inbound/rev/3ff0432ce8d9 https://hg.mozilla.org/integration/mozilla-inbound/rev/2373c969584c
Comment 10•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/9679ec5b0b33 https://hg.mozilla.org/mozilla-central/rev/fb81587d2dd8 https://hg.mozilla.org/mozilla-central/rev/3ff0432ce8d9 https://hg.mozilla.org/mozilla-central/rev/2373c969584c
Status: NEW → RESOLVED
Closed: 12 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
You need to log in
before you can comment on or make changes to this bug.
Description
•