Closed Bug 678069 Opened 10 years ago Closed 10 years ago
Fix c++0x initializer list error found by clang
Fix build with clang and c++0x. mShWidth and mShHeight are used in an initializer list for NPRect in fields that have time uint16_t. Change their definitions to match.
Attachment #552231 - Flags: review?(mh+mozilla)
Attachment #552231 - Flags: review?(mh+mozilla) → review?(joshmoz)
Comment on attachment 552231 [details] [diff] [review] fix build mShWidth and mShHeight are also used for "CGBitmapContextCreate", which wants size_t, and "CGRectMake", which wants CGFloat. Is the logic here to make the stored value the smallest of all the requirements?
Almost. It is one that will not lose a value when going to uint16_t in a initializer list (which is the c++0x restriction).
Assignee: nobody → respindola
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
Version: unspecified → Trunk
You need to log in before you can comment on or make changes to this bug.