Closed Bug 1487384 Opened 2 years ago Closed 2 years ago

Tidy unnecessary use of |rt->gc| in GCRuntime methods

Categories

(Core :: JavaScript: GC, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: jonco, Assigned: jonco)

Details

Attachments

(1 file)

No description provided.
This is probably a hangover from when this code was all static functions taking a JSRuntime*.
Summary: Tidy unnecessary use of → Tidy unnecessary use of |rt->gc| in GCRuntime methods
This also does a bit of refactoring to remove rt->gc in a couple of other places, like making the background sweep task call a GCRuntime method to do the work.
Attachment #9005187 - Flags: review?(sphink)
Attachment #9005187 - Flags: review?(sphink) → review+
Pushed by jcoppeard@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/8a2005a771b3
Remove unnecessary references to Runtime::gc in GCRuntime methods r=sfink
https://hg.mozilla.org/mozilla-central/rev/8a2005a771b3
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.