Closed
Bug 397533
Opened 17 years ago
Closed 17 years ago
Remove some useless includes/instance definitions of nsISupportsArray in mailnews.
Categories
(MailNews Core :: Backend, defect)
MailNews Core
Backend
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: standard8, Assigned: standard8)
References
Details
Attachments
(1 file)
|
(deleted),
patch
|
Bienvenu
:
review+
Bienvenu
:
superreview+
|
Details | Diff | Splinter Review |
There's some includes of nsISupportsArray.h and some instance definitions for the class that are just redundant and not required. We should remove them and start helping the remove nsISupportsArray effort ;-)
As the content of the interfaces in the idls isn't changing, we don't have to change uuids.
Attachment #282299 -
Flags: superreview?(bienvenu)
Attachment #282299 -
Flags: review?(bienvenu)
|
||
Comment 1•17 years ago
|
||
Comment on attachment 282299 [details] [diff] [review]
The fix
as long as it still compiles :-)
Attachment #282299 -
Flags: superreview?(bienvenu)
Attachment #282299 -
Flags: superreview+
Attachment #282299 -
Flags: review?(bienvenu)
Attachment #282299 -
Flags: review+
|
Assignee | |
Comment 2•17 years ago
|
||
Checked in -> fixed.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
what does this mean for things like, for example, getDragData in nsDragAndDrop.js - will there be some new interface to replace nsISupportsArray functionality?
|
|
Assignee | |
Comment 4•17 years ago
|
||
(In reply to comment #3)
> what does this mean for things like, for example, getDragData in
> nsDragAndDrop.js - will there be some new interface to replace nsISupportsArray
> functionality?
See http://wiki.mozilla.org/Gecko:Obsolete_API and http://mxr.mozilla.org/seamonkey/source/xpcom/ds/nsISupportsArray.idl#42 - nsISupportsArray is obsolete. It will most likely be replaced by an nsIArray interface as thats the most similar.
|
||
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•