Closed Bug 1045940 Opened 6 years ago Closed 6 years ago

Rename needsBarrier to needsIncrementalBarrier

Categories

(Core :: JavaScript: GC, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla34

People

(Reporter: terrence, Assigned: terrence)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

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)
Attachment #8464367 - Flags: review?(jcoppeard) → review+
https://hg.mozilla.org/mozilla-central/rev/b58d03207432
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
Flags: qe-verify-
You need to log in before you can comment on or make changes to this bug.