Closed
Bug 26420
Opened 25 years ago
Closed 25 years ago
Enable |ASSERT_NO_QUERY_NEEDED| for the |getter_AddRefs()| case
Categories
(Core :: XPCOM, defect, P3)
Core
XPCOM
Tracking
()
VERIFIED
FIXED
M14
People
(Reporter: scc-obsolete, Assigned: scc-obsolete)
Details
When assigning into an |nsCOMPtr| using |=| and the user didn't specify |
do_QueryInterface| or provide another |nsCOMPtr| of the same type, a test is
performed after the assignment to ensure that no query was actually needed. But
there is another way to assign into an |nsCOMPtr|, you can pass it as a `result
parameter' with |getter_AddRefs()|. In a debug build, |getter_AddRefs()| should
perform the same test.
I currently can't enable this test because the assert fires twice for each page
load. The underlying bug is that proxies don't correctly support |
QueryInterface|. <dougt@netscape.com> has code to fix this, when he checks that
in, the test can be enabled.
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M14
Assignee | ||
Comment 1•25 years ago
|
||
Alright. <dougt@netscape.com> checked in his fixes that allow me to turn this
on, so I'm turning it on now.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•