(Comment) Cleanup after bug 279210

NEW
Unassigned

Status

()

--
minor
10 years ago
a year ago

People

(Reporter: sgautherie, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

10 years ago
Bug 279210 comment 7:
{{
David Baron [:dbaron]   2008-08-07 14:37:33 PDT

(From update of attachment 332753 [details] [diff] [review])
OK.  I've never been crazy about nsDerivedSafe, since it requires extra casts
or .get() when static_casting an nsCOMPtr to a more-derived class; we should
probably file a followup bug on finding and removing those...
}}

I don't know if these (comment) locations are what David was thinking about;
let's do it/both...

Comment 1

10 years ago
Dehydra could certainly give us a list of places which call nsCOMPtr.get()... I'm not sure whether it has the ability to give us a list of places we can "safely" remove the .get(), because differences between the different cast styles are I think dealt with in the parser, before dehydra sees them.

Comment 2

10 years ago
Oh, I didn't see the URL link... it would certainly be worth looking at those, but I don't think that's the only places dbaron was talking about... I'm not sure, though, TBH.
You need to log in before you can comment on or make changes to this bug.