Closed Bug 1447284 Opened 2 years ago Closed 2 years ago

The comments above AutoSuppressGC are out of date

Categories

(Core :: JavaScript: GC, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla61
Tracking Status
firefox61 --- fixed

People

(Reporter: jonco, Assigned: jonco)

Details

Attachments

(1 file)

They refer to JSRuntime::suppressGC which has moved to JSContext.
Priority: -- → P3
Patch to fix comments to refer to JSContext rather than JSRuntime.
Assignee: nobody → jcoppeard
Attachment #8968480 - Flags: review?(pbone)
Comment on attachment 8968480 [details] [diff] [review]
bug1447284-fix-comments

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

Yep. This is now correct, and also explained a little better  Thanks.
Attachment #8968480 - Flags: review?(pbone) → review+
Pushed by jcoppeard@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/af53333e6e32
Fix comments on AutoSuppressGC class r=pbone
https://hg.mozilla.org/mozilla-central/rev/af53333e6e32
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
You need to log in before you can comment on or make changes to this bug.