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

RESOLVED FIXED in mozilla8

Status

()

Toolkit
Crash Reporting
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: espindola, Assigned: espindola)

Tracking

Trunk
mozilla8
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Created attachment 551848 [details] [diff] [review]
add cast

The process_id field is unsigned, so we need this cast in c++0x. Let me know if you would like me to try to make the field signed instead.
See http://llvm.org/viewvc/llvm-project/?view=rev&revision=136181
for more information.
Attachment #551848 - Flags: review?(mh+mozilla)
Attachment #551848 - Flags: review?(mh+mozilla) → review?(ted.mielczarek)

Comment 1

6 years ago
Committed Breakpad r825.
Assignee: nobody → respindola
Comment on attachment 551848 [details] [diff] [review]
add cast

Feel free to land this in m-c as well.
Attachment #551848 - Flags: review?(ted.mielczarek) → review+
Keywords: checkin-needed
http://hg.mozilla.org/integration/mozilla-inbound/rev/2012711170aa

It looks like the commit message in the attached patch included a chunk of the bug description. ("The process_id field is unsigned, so we need this cast in c++0x. Let me know if you would like me to try to make the field signed instead.
See http://llvm.org/viewvc/llvm-project/?view=rev&revision=136181
for more information.")

I trimmed it, but it'd be helpful if you could make sure the commit message in checkin-needed patches was more commit-message-ish. :)
Keywords: checkin-needed
Whiteboard: [inbound]
Target Milestone: --- → mozilla8
Version: unspecified → Trunk
https://hg.mozilla.org/mozilla-central/rev/2012711170aa
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
You need to log in before you can comment on or make changes to this bug.