Closed Bug 1640977 Opened 3 months ago Closed 2 months ago

Tidy ArenaLists class definition

Categories

(Core :: JavaScript: GC, task)

Product:
Core
Component:
JavaScript: GC
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla78
Tracking Flags:
Tracking Status
firefox78 --- fixed

People

(Reporter: jonco, Assigned: jonco)

Details

Attachments

(2 files)

Bug 1640977 - Rename some ArenaLists members for clarity r?sfink
47 bytes, text/x-phabricator-request
Details | Review
Bug 1640977 - Tidy the ArenaLists class definition to put all data members in one place r?sfink
47 bytes, text/x-phabricator-request
Details | Review

While working on bug 1508327 I made some small tidyups to the ArenaLists class.

The arenaLists() method returns an ArenaList, confusing when we also have an ArenaLists class, and arenaListsToSweep() returns an Arena. This gives them clearer names.

Pushed by jcoppeard@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2348767fd355
Rename some ArenaLists members for clarity r=sfink
https://hg.mozilla.org/integration/autoland/rev/ff13d34e8f77
Tidy the ArenaLists class definition to put all data members in one place r=sfink

https://hg.mozilla.org/mozilla-central/rev/2348767fd355
https://hg.mozilla.org/mozilla-central/rev/ff13d34e8f77

Status: NEW → RESOLVED
Closed: 2 months ago
status-firefox78: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla78
You need to log in before you can comment on or make changes to this bug.