Closed Bug 735615 Opened 12 years ago Closed 12 years ago

Remove AutoClean from mozglue/linker/Utils.h

Categories

(Core :: mozglue, defect)

ARM
Android
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla17

People

(Reporter: Yoric, Assigned: drexler)

References

Details

(Whiteboard: [mentor=Yoric][lang=c++])

Attachments

(1 file)

Now that we have generic RAII helpers in MFBT, we should get rid of the corresponding original code in mozglue/linker/Utils.h, which does not compile with VC++.
Note: to test that it does not break faulty.lib, add |export MOZ_LINKER=1| to mozconfig and build on Linux desktop.
Actually, it looks like a good candidate for mentored bugs.
Component: XPCOM → XP Toolkit/Widgets: Menus
QA Contact: xpcom → xptoolkit.menus
Whiteboard: [mentor=Yoric][lang=c++]
Component: XP Toolkit/Widgets: Menus → mozglue
OS: Mac OS X → Android
QA Contact: xptoolkit.menus → mozglue
Hardware: x86 → ARM
QA Contact: andrew.quartey
Assignee: nobody → andrew.quartey
QA Contact: andrew.quartey
Attached patch patchSplinter Review
Attachment #646882 - Flags: review?(dteller)
QA Contact: mozglue
Comment on attachment 646882 [details] [diff] [review]
patch

I'll take this review
Attachment #646882 - Flags: review?(dteller) → review?(mh+mozilla)
Attachment #646882 - Flags: review?(mh+mozilla) → review+
Sent to inbound: https://hg.mozilla.org/integration/mozilla-inbound/rev/3c34aa0c81f7
Status: NEW → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/3c34aa0c81f7
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
Depends on: 826173
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: