Closed
Bug 749366
Opened 12 years ago
Closed 12 years ago
Remove unused and mostly bogus size argument from Describe{RefCounted,GCed}Node CC callback
Categories
(Core :: XPCOM, defect)
Core
XPCOM
Tracking
()
RESOLVED
FIXED
mozilla18
People
(Reporter: mccr8, Assigned: mccr8)
References
(Blocks 1 open bug)
Details
Attachments
(2 files, 1 obsolete file)
22.03 KB,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
1.68 KB,
patch
|
n.nethercote
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•12 years ago
|
||
I should probably remove the renaming I did of the appallingly named "foo" local variable.
Assignee: nobody → continuation
Assignee | ||
Comment 2•12 years ago
|
||
Attachment #618848 -
Attachment is obsolete: true
Assignee | ||
Comment 3•12 years ago
|
||
Assignee | ||
Comment 4•12 years ago
|
||
Comment on attachment 655379 [details] [diff] [review]
part 1: remove the arguments
Try run looked okay. Kind of a shame we don't have some better way to encapsulate Traverses that want to specify their own name.
https://tbpl.mozilla.org/?tree=Try&rev=9dd920702889
Attachment #655379 -
Flags: review?(bugs)
Assignee | ||
Comment 5•12 years ago
|
||
Comment on attachment 655380 [details] [diff] [review]
part 2: remove the now unused JSContextSize
This function was added by Ms2ger to remove jscntxt.h (or whatever) from the exported JS headers, but it isn't needed any more after this patch, so it makes sense to remove it, I think.
Attachment #655380 -
Flags: review?(n.nethercote)
Updated•12 years ago
|
Attachment #655380 -
Flags: review?(n.nethercote) → review+
Updated•12 years ago
|
Attachment #655379 -
Flags: review?(bugs) → review+
Assignee | ||
Comment 6•12 years ago
|
||
Comment 7•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/61efc4bbf95b
https://hg.mozilla.org/mozilla-central/rev/65e30427c102
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
You need to log in
before you can comment on or make changes to this bug.
Description
•