In xpcom/threads/nsEventQueueService.cpp there is this: /* nsISupports interface implementation... */ NS_IMPL_ISUPPORTS(EventQueueEntry,nsISupports::GetIID()); (depending on when look, it might be using kISupportsIID instead of nsISupports::GetIID(). Either way, why is this implementing a QueryInterface for nsISupports rather than one for EventQueueEntry?
Btw, the reason I might have changed the kISupportsIID to nsISupports::GetIID() is that I'm working on removing the kISupportsIID static declaration from the default implementation of QueryInterface, and this made an error with kISupportsIID not being defined here. (Just an FYI on the change.)
Severity: normal → major
Status: NEW → ASSIGNED
Priority: P3 → P2
Target Milestone: M7
Good point. Will fix it.
Well nsISupports is right as it is using nsISupport not to implement an interface but to do refcounting. Interesting...
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → INVALID
Okay, then I will go and apply the patch that I have on #7284 to change this to nsISupports::GetIID() when the rest of the patches get applied. Thanks!
You need to log in before you can comment on or make changes to this bug.