Closed Bug 329879 Opened 14 years ago Closed 14 years ago

Coverity complains that QName null checks argv after asserting it can't be null

Categories

(Core :: JavaScript Engine, defect, trivial)

PowerPC
macOS
defect
Not set
trivial

Tracking

()

RESOLVED FIXED

People

(Reporter: timeless, Assigned: timeless)

References

(Blocks 1 open bug, )

Details

(Keywords: coverity)

Attachments

(1 obsolete file)

I'm pretty sure this can't be null, but i don't understand why brendan changed the code in 3.11 ...
Attached patch revert unreachable codepath (obsolete) — Splinter Review
Attachment #214533 - Flags: review?(brendan)
Comment on attachment 214533 [details] [diff] [review]
revert unreachable codepath

Hmm, why'd I do that?  The voluminous checkin comments don't say.  Oh well.  Thanks,

/be
Attachment #214533 - Flags: review?(brendan) → review+
Comment on attachment 214533 [details] [diff] [review]
revert unreachable codepath

mozilla/js/src/jsxml.c 	3.91

yeah, it was such a long comment, and it didn't answer the question i needed answered. oh well :)
Attachment #214533 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.