Closed Bug 408731 Opened 17 years ago Closed 17 years ago

Remove nsISupportsArray usage from nsISubscribableServer::GetChildren

Categories

(MailNews Core :: Backend, defect)

Product:
MailNews Core
Component:
Backend
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(1 file)

The fix
(deleted), patch
Bienvenu
: review+
Bienvenu
: superreview+
Details | Diff | Splinter Review
Attached patch The fix (deleted) — — Splinter Review
Looking at the code, rather than creating an nsISupportsArray and passing it back to the main function that uses it to create an nsISimpleEnumerator, its just simpler to create an nsISimpleEnumerator at the source of the data and pass it all the way back to the caller. We can also then easily replace nsISupportsArray with nsCOMArray internally which makes things easier.
Attachment #293574 - Flags: superreview?(bienvenu)
Attachment #293574 - Flags: review?(bienvenu)
Comment on attachment 293574 [details] [diff] [review] The fix thx, Mark
Attachment #293574 - Flags: superreview?(bienvenu)
Attachment #293574 - Flags: superreview+
Attachment #293574 - Flags: review?(bienvenu)
Attachment #293574 - Flags: review+
Patch checked in -> fixed.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: