Last Comment Bug 462083 - add nsIArray support to mailnews/base/util/iteratorUtils.jsm
: add nsIArray support to mailnews/base/util/iteratorUtils.jsm
[fixed by bug 698925]
Product: MailNews Core
Classification: Components
Component: Backend (show other bugs)
: Trunk
: All All
-- normal (vote)
: Thunderbird 12.0
Assigned To: Nobody; OK to take it and work on it
Depends on: 418490 BigFiles
  Show dependency treegraph
Reported: 2008-10-28 17:57 PDT by David :Bienvenu
Modified: 2012-11-19 22:36 PST (History)
4 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---


Description User image David :Bienvenu 2008-10-28 17:57:58 PDT
spun-off from bug 418490, toXPCOMIterator should handle nsIArray as well.
Comment 1 User image David :Bienvenu 2008-10-31 16:58:12 PDT
I'm not sure this is needed - the js code needs to create an nsIMutableArray in order to put things in the array, and if you pass in the nsIMutableArray interface, you can pass the result to callers expecting an nsIArray.
Comment 2 User image 2008-10-31 17:12:14 PDT
The point is that the JS can create a normal array and then use the utility to convert it to an nsIArray, and can also use the utility to iterate an nsIArray.
Comment 3 User image David :Bienvenu 2008-10-31 17:24:30 PDT
iterating an nsIArray makes sense, but converting it to an nsIArray would mean creating an nsIMutableArray, and then I guess QI'ing that to an nsIArray, right?
Comment 4 User image 2008-11-01 06:47:46 PDT
nsIMutableArray inherits from nsIArray last time I looked.
Comment 5 User image :aceman 2012-11-19 14:09:08 PST
Fixed in bug 698925.

*** This bug has been marked as a duplicate of bug 698925 ***

Note You need to log in before you can comment on or make changes to this bug.