Closed
Bug 211196
Opened 22 years ago
Closed 22 years ago
[FIX]Clean up some array stuff in nsXMLContentSink
Categories
(Core :: XML, defect)
Core
XML
Tracking
()
RESOLVED
FIXED
People
(Reporter: bzbarsky, Assigned: bzbarsky)
Details
Attachments
(1 file, 1 obsolete file)
|
18.03 KB,
patch
|
caillon
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
Making things clearer as to what's addrefed and what's not.
|
Assignee | |
Comment 1•22 years ago
|
||
|
|
Assignee | |
Updated•22 years ago
|
Attachment #126805 -
Flags: superreview?(jst)
Attachment #126805 -
Flags: review?(caillon)
|
||
Comment 2•22 years ago
|
||
Comment on attachment 126805 [details] [diff] [review]
patch
sr=jst
Attachment #126805 -
Flags: superreview?(jst) → superreview+
|
|
Assignee | |
Comment 3•22 years ago
|
||
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
|
|
Assignee | |
Updated•22 years ago
|
Attachment #126805 -
Flags: review?(caillon)
|
|
Assignee | |
Updated•22 years ago
|
Attachment #126881 -
Flags: superreview?(jst)
Attachment #126881 -
Flags: review?(caillon)
|
|
||
Comment 4•22 years ago
|
||
Comment on attachment 126881 [details] [diff] [review]
Better patch
sr=jst
Attachment #126881 -
Flags: superreview?(jst) → superreview+
|
|
||
Updated•22 years ago
|
Attachment #126881 -
Flags: review?(caillon) → review+
|
|
Assignee | |
Comment 5•22 years ago
|
||
Fixed.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•