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

Status: RESOLVED FIXED

(Core :: General, defect)

Description

[Rafael Ávila de Espíndola (:espindola) (not reading bugmail)]

Fix build with clang and c++0x.
    
    Use tighter types for some fields. These fields are used in
    
      CFGregorianDate gregorian = {
        exploded.year,
        exploded.month,
        exploded.day_of_month,
        exploded.hour,
        exploded.minute,
        seconds
      };
    
    and in CFGregorianDate the month, day, hour and minute fields are 8 bits. Let
    me know if I should also change the day_of_week and second fields.

Comment 1

[Rafael Ávila de Espíndola (:espindola) (not reading bugmail)]

Attachment: fix build

Comment 2

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Comment on attachment 552391 [details] [diff] [review]
fix build

Yes, please change those fields too.

Comment 3

[Rafael Ávila de Espíndola (:espindola) (not reading bugmail)]

Attachment: updated patch

Comment 4

[(no longer active)]

Pushed to inbound.

Comment 5

[Matt Brubeck (:mbrubeck)]

https://hg.mozilla.org/mozilla-central/rev/5812274f4afb