Remove some useless includes/instance definitions of nsISupportsArray in mailnews.

RESOLVED FIXED

Status

MailNews Core
Backend
RESOLVED FIXED
11 years ago
10 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

11 years ago
Created attachment 282299 [details] [diff] [review]
The fix

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

11 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

11 years ago
Checked in -> fixed.
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED

Comment 3

11 years ago
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

11 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.
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.