Closed Bug 1411636 Opened 2 years ago Closed 2 years ago

Remove unnecessary recalculation of GCRuntime::isFull

Categories

(Core :: JavaScript: GC, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla58
Tracking Status
firefox58 --- fixed

People

(Reporter: jonco, Assigned: jonco)

Details

Attachments

(1 file)

We used to recalculate whether we were doing a full GC at the end of sweeping, because we had to pass this information to a callback.  We removed this argument a while ago so the recalculation is no longer required.
Attachment #8921945 - Flags: review?(sphink)
Comment on attachment 8921945 [details] [diff] [review]
bug1411636-remove-is-full-recalc

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

Cool, nice to lose a loop through the zones.
Attachment #8921945 - Flags: review?(sphink) → review+
Pushed by jcoppeard@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/675b7b1772d1
Remove unnecessary recalculation of GCRuntime::isFull r=sfink
https://hg.mozilla.org/mozilla-central/rev/675b7b1772d1
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.