Closed
Bug 402612
Opened 17 years ago
Closed 17 years ago
Reenable MOZ_COUNT_[CD]TOR in CToken and ~CToken
Categories
(Core :: DOM: HTML Parser, defect)
Core
DOM: HTML Parser
Tracking
()
RESOLVED
WONTFIX
mozilla1.9beta2
People
(Reporter: Waldo, Assigned: Waldo)
Details
Attachments
(1 file)
(deleted),
patch
|
sicking
:
review+
sicking
:
superreview+
|
Details | Diff | Splinter Review |
I really don't know why these were commented out, because even if CToken isn't refcounted this is still useful for detecting when CTokens are leaked (as currently happens with the currently-remaining nsStringBuffer leaks in bug 395845). I suspect whoever did this was just confused about how these macros are actually supposed to be used.
Attachment #287460 -
Flags: superreview?(mrbkap)
Attachment #287460 -
Flags: review?(mrbkap)
Attachment #287460 -
Flags: superreview?(mrbkap)
Attachment #287460 -
Flags: superreview+
Attachment #287460 -
Flags: review?(mrbkap)
Attachment #287460 -
Flags: review+
Assignee | ||
Updated•17 years ago
|
Attachment #287460 -
Flags: approval1.9?
Comment 1•17 years ago
|
||
I wonder if this was commented out to handle the case of purposely leaving tokens allocated and letting the destruction of their arena deallocate them. That's bogus, of course, since you always have to run destructors but it's the only explanation that I can come up with.
Comment 2•17 years ago
|
||
I don't think this is the way to go. Since tokens are refcounted, I think we should log the refcounting. That's basically what I'm doing for the patch in bug 402633.
Assignee | ||
Comment 3•17 years ago
|
||
Oh, I didn't look far enough to see that -- I wasn't really expecting parser tokens were refcounted, to be honest. Given that, yeah, this is clearly WONTFIX.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → WONTFIX
Assignee | ||
Updated•17 years ago
|
Attachment #287460 -
Flags: approval1.9?
You need to log in
before you can comment on or make changes to this bug.
Description
•