Closed Bug 1252902 Opened 4 years ago Closed 4 years ago

remove ScopedFreePtr

Categories

(Core :: MFBT, defect)

Product:
Core
Component:
MFBT
Type:
defect
Priority:
Not set

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla47
Tracking Flags:
Tracking Status
firefox47 --- fixed

People

(Reporter: froydnj, Assigned: froydnj)

Details

Attachments

(1 file)

remove ScopedFreePtr
4.08 KB, patch
Waldo
: review+
Details | Diff | Splinter Review 
We have UniqueFreePtr as a replacement, which at least uses
more-standard machinery to do its job.
This patch depends on in-flight patches, but the basic idea is sound.  A lot of
the patch is simply reformatting the header comment to make more sense in light
of the header only providing macros now.  Removing Scoped.h entirely comes later.
Attachment #8725731 - Flags: review?(jwalden+bmo)
Attachment #8725731 - Flags: review?(jwalden+bmo) → review+
https://hg.mozilla.org/mozilla-central/rev/7cace407920c
Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox47: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
You need to log in before you can comment on or make changes to this bug.