Closed Bug 678060 Opened 10 years ago Closed 10 years ago

Fix c++0x initializer list error found by clang

Categories

(Core :: Widget: Cocoa, defect)

All
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla8

People

(Reporter: espindola, Unassigned)

Details

Attachments

(1 file)

Fix build with clang and c++0x.
    ThemeMenuState is defined with a typedef from an integer type, not as an
    enum. Because of that, we need to add explicit casts when "assigning" to
    it in an initializer list.
Attached patch fix buildSplinter Review
Attachment #552218 - Flags: review?(mh+mozilla)
Attachment #552218 - Flags: review?(mh+mozilla) → review?(joshmoz)
Attachment #552218 - Flags: review?(joshmoz) → review+
http://hg.mozilla.org/integration/mozilla-inbound/rev/059713859dcc
Status: NEW → ASSIGNED
Keywords: checkin-needed
Whiteboard: [inbound]
Target Milestone: --- → mozilla8
Version: unspecified → Trunk
OS: Linux → Mac OS X
Hardware: x86_64 → All
https://hg.mozilla.org/mozilla-central/rev/059713859dcc
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
You need to log in before you can comment on or make changes to this bug.