Closed Bug 333354 Opened 14 years ago Closed 14 years ago

mlk on early return

Categories

(Core :: Layout: Tables, defect)

x86
Windows XP
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: bernd_mozilla, Assigned: bernd_mozilla)

References

(Blocks 1 open bug, )

Details

(Keywords: coverity, memory-leak)

Attachments

(1 file, 1 obsolete file)

1.88 KB, patch
bzbarsky
: review+
bzbarsky
: superreview+
Details | Diff | Splinter Review
coverity complains about it.
Attached patch patch (obsolete) — Splinter Review
Attachment #217768 - Flags: superreview?(bzbarsky)
Attachment #217768 - Flags: review?(bzbarsky)
Why not just use nsAutoArrayPtr here instead and not have to deal with manaul deletes at all?
Keywords: coverity, mlk
Comment on attachment 217768 [details] [diff] [review]
patch

AutoArrayPtr, please!  ;)
Attachment #217768 - Flags: superreview?(bzbarsky)
Attachment #217768 - Flags: superreview-
Attachment #217768 - Flags: review?(bzbarsky)
Attachment #217768 - Flags: review-
Attached patch autoptrSplinter Review
Attachment #217768 - Attachment is obsolete: true
Attachment #223063 - Flags: superreview?(bzbarsky)
Attachment #223063 - Flags: review?(bzbarsky)
Attachment #223063 - Flags: superreview?(bzbarsky)
Attachment #223063 - Flags: superreview+
Attachment #223063 - Flags: review?(bzbarsky)
Attachment #223063 - Flags: review+
fix checked in
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.