Closed Bug 831879 Opened 7 years ago Closed 7 years ago

Disable "maybe used uninitialized" warnings in dom/bindings

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla22

People

(Reporter: Ms2ger, Assigned: Ms2ger)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch Patch v1 (obsolete) — Splinter Review
It's not like we care about them.
Attachment #703440 - Flags: review?(khuey)
Comment on attachment 703440 [details] [diff] [review]
Patch v1

Review of attachment 703440 [details] [diff] [review]:
-----------------------------------------------------------------

This may surprise you but there are compilers other than gcc.
Attachment #703440 - Flags: review?(khuey) → review-
It does surprise me. Would you take the patch with some kind of ifdef, and if so, what kind?
Flags: needinfo?(khuey)
ifdef GNU_CC would be a good start.

To be clear, I'm not just picking nits, the patch as is won't compile on Windows.
Flags: needinfo?(khuey)
Attached patch Patch v2Splinter Review
Attachment #703440 - Attachment is obsolete: true
Attachment #719914 - Flags: review?(khuey)
https://hg.mozilla.org/mozilla-central/rev/7100b423c624
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
Depends on: 1240592
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.