Closed Bug 677648 Opened 8 years ago Closed 8 years ago

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

Categories

(Core :: Layout, defect)

x86_64
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla8

People

(Reporter: espindola, Assigned: espindola)

Details

Attachments

(1 file)

The mAdvance field of DetailedGlyph is a PRInt32. Since the advance
    variable is used in
    
    DetailedGlyph detail = {glyphIndex, advance, 0, 0};
    
    it should be a PRInt32 instead of a PRUint32. Let me know if I should try
    to change the field instead.
Attachment #551857 - Flags: review?(mh+mozilla)
Attachment #551857 - Flags: review?(mh+mozilla) → review?(roc)
Assignee: nobody → respindola
http://hg.mozilla.org/mozilla-central/rev/90719558b5e2
Status: NEW → RESOLVED
Closed: 8 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
Version: unspecified → Trunk
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.