Last Comment Bug 685593 - remove unused argument gcKind from Debugger::markAllIteratively
: remove unused argument gcKind from Debugger::markAllIteratively
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: Trunk
: All All
-- normal (vote)
: mozilla9
Assigned To: Andrew McCreight [:mccr8]
: Jason Orendorff [:jorendorff]
Depends on:
Blocks: 653248
  Show dependency treegraph
Reported: 2011-09-08 10:33 PDT by Andrew McCreight [:mccr8]
Modified: 2011-09-10 16:15 PDT (History)
4 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

remove the argument (2.85 KB, patch)
2011-09-08 10:47 PDT, Andrew McCreight [:mccr8]
jorendorff: review+
Details | Diff | Splinter Review

Description User image Andrew McCreight [:mccr8] 2011-09-08 10:33:34 PDT
The argument gcKind is unused, and it makes moving markAllIteratively out of MarkAndSweep into MarkRuntime, which I am considering doing as part of some GC cleanup, impossible.
Comment 1 User image Andrew McCreight [:mccr8] 2011-09-08 10:47:43 PDT
Created attachment 559216 [details] [diff] [review]
remove the argument

I haven't tested this yet, but it compiles, so there shouldn't be a problem.  I'll try server it before I land it.
Comment 2 User image Andrew McCreight [:mccr8] 2011-09-08 16:38:07 PDT
Comment 3 User image Mozilla RelEng Bot 2011-09-08 23:50:55 PDT
Try run for 51307568846d is complete.
Detailed breakdown of the results available here:
Results (out of 146 total builds):
    exception: 2
    success: 141
    warnings: 2
    failure: 1
Builds available at
Comment 5 User image Mike Shaver (:shaver -- probably not reading bugmail closely) 2011-09-10 16:15:46 PDT
There is still no actual alliteration in this function, but I guess that's a separate bug.

Note You need to log in before you can comment on or make changes to this bug.