Closed Bug 717220 Opened 9 years ago Closed 9 years ago

Tons of 'comma at end of enumerator list' warnings

Categories

(Core :: IPC, defect)

x86
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla12

People

(Reporter: jacek, Assigned: jacek)

Details

Attachments

(1 file)

Attached patch fix v1.0Splinter Review
It's very verbose on my mingw builds. The attached patch fixes all of them related to ipc/ sources.
Attachment #587672 - Flags: review?(dholbert)
Comment on attachment 587672 [details] [diff] [review]
fix v1.0

Patch looks good to me! I don't have any sort of ownership over ipc, though -- you should get sign-off from someone who's touched that directory more than I have.

Kicking over to cjones.
Attachment #587672 - Flags: review?(jones.chris.g)
Attachment #587672 - Flags: review?(dholbert)
Attachment #587672 - Flags: feedback+
Attachment #587672 - Flags: review?(jones.chris.g) → review+
Thanks for reviews, pushed to m-i:

https://hg.mozilla.org/integration/mozilla-inbound/rev/f357bdd35f1b
Whiteboard: [inbound]
https://hg.mozilla.org/mozilla-central/rev/f357bdd35f1b
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla12
You need to log in before you can comment on or make changes to this bug.