Unnecessary "not derived from nsISupports" warnings for XPCOM pseudo-enums
RESOLVED
FIXED
in mozilla12
Status
()
People
(Reporter: Irving, Assigned: Irving)
Tracking
Bug Flags:
Firefox Tracking Flags
(Not tracked)
Details
Attachments
(1 attachment)
|
1.56 KB,
patch
|
khuey
:
review+
|
Details | Diff | Splinter Review |
Thunderbird has many IDL definitions of pseudo-enums, basically an IDL interface that only declares a bunch of constants. Because we never create an instance of the IDL object, it doesn't need any of the normal nsISupports infrastructure.
xpcom prints warnings about all of these, like:
warning: interface 'nsMsgPriority' not derived from nsISupports, /Users/ireid/tbird/comm-central/mailnews/base/public/MailNewsTypes2.idl line 52:0
interface nsMsgPriority {
^
Based on a conversation on irc #developers, this warning doesn't add much value and can be removed.
|
(Assignee) | |
Comment 1•7 years ago
|
||
Created attachment 572912 [details] [diff] [review] Remove 'not derived from nsISupports' warning
Attachment #572912 -
Flags: review?(khuey)
|
|
(Assignee) | |
Updated•7 years ago
|
||
Assignee: nobody → irving
Attachment #572912 -
Flags: review?(khuey) → review+
|
|
(Assignee) | |
Comment 2•7 years ago
|
||
I don't have mozilla-central checkin privileges; anyone ready to take this on?
Status: NEW → ASSIGNED
Keywords: checkin-needed
|
||
Comment 3•7 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/8e7225a8b1cc
Flags: in-testsuite-
Keywords: checkin-needed
Target Milestone: --- → mozilla12
|
||
Comment 4•7 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/8e7225a8b1cc
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•