Closed
Bug 1045940
Opened 10 years ago
Closed 10 years ago
Rename needsBarrier to needsIncrementalBarrier
Categories
(Core :: JavaScript: GC, defect)
Core
JavaScript: GC
Tracking
()
RESOLVED
FIXED
mozilla34
People
(Reporter: terrence, Assigned: terrence)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
|
(deleted),
patch
|
jonco
:
review+
|
Details | Diff | Splinter Review |
This is a bit confusing: the incremental isBarrierNeeded flag is public in JS::shadow::Runtime so that we can inline it for ExposeToActiveJS and MarkGCThingAsLive, yet the only actual barriers we expose to the browser are ggc's post barriers. I've renamed needsBarrier to needsIncrementalBarrier to hopefully make this clearer.
Attachment #8464367 -
Flags: review?(jcoppeard)
|
||
Updated•10 years ago
|
Attachment #8464367 -
Flags: review?(jcoppeard) → review+
|
Assignee | |
Comment 1•10 years ago
|
||
|
||
Comment 2•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
|
|
||
Comment 3•10 years ago
|
||
|
||
Updated•10 years ago
|
Flags: qe-verify-
You need to log in
before you can comment on or make changes to this bug.
Description
•