Closed
Bug 1162590
Opened 9 years ago
Closed 9 years ago
Give the 2-arg traverse a different name
Categories
(Core :: JavaScript: GC, defect)
Core
JavaScript: GC
Tracking
()
RESOLVED
FIXED
mozilla40
Tracking | Status | |
---|---|---|
firefox40 | --- | fixed |
People
(Reporter: terrence, Assigned: terrence)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
12.79 KB,
patch
|
sfink
:
review+
|
Details | Diff | Splinter Review |
At first I meant this as an optional wrapper, but seeing that it gives us extra assertions I'd like to see us use it unconditionally for internal edges. This will be easier to verify via search if we give it a different name, at least.
Attachment #8602795 -
Flags: review?(sphink)
Comment 1•9 years ago
|
||
Comment on attachment 8602795 [details] [diff] [review] 23_give_traceEdge_a_different_name-v0.diff Review of attachment 8602795 [details] [diff] [review]: ----------------------------------------------------------------- Okey dokie.
Attachment #8602795 -
Flags: review?(sphink) → review+
Assignee | ||
Comment 2•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=fc4313407098
Comment 4•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/428de38c2edc
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in
before you can comment on or make changes to this bug.
Description
•