Closed Bug 211196 Opened 21 years ago Closed 21 years ago

[FIX]Clean up some array stuff in nsXMLContentSink

Categories

(Core :: XML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: bzbarsky, Assigned: bzbarsky)

Details

Attachments

(1 file, 1 obsolete file)

Making things clearer as to what's addrefed and what's not.
Attached patch patch (obsolete) — Splinter Review
Attachment #126805 - Flags: superreview?(jst)
Attachment #126805 - Flags: review?(caillon)
Comment on attachment 126805 [details] [diff] [review]
patch

sr=jst
Attachment #126805 - Flags: superreview?(jst) → superreview+
Attached patch Better patchSplinter Review
Unfortunately, XBL triggered that assert that I added, in spades.... And I had
to make a minot change to the XBL sink to fix some crashes I ran into.
Attachment #126805 - Attachment is obsolete: true
Attachment #126805 - Flags: review?(caillon)
Attachment #126881 - Flags: superreview?(jst)
Attachment #126881 - Flags: review?(caillon)
Comment on attachment 126881 [details] [diff] [review]
Better patch

sr=jst
Attachment #126881 - Flags: superreview?(jst) → superreview+
Attachment #126881 - Flags: review?(caillon) → review+
Fixed.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: