1682945 - Replace idl nsISimpleEnumerator usage with Array<T> in mailnews/search/

Status: RESOLVED FIXED

(MailNews Core :: General, task)

Description

[Ben Campbell]

Replacing nsISimpleEnumerator with Array<> (where sensible!)

This covers:
mailnews/search/public/nsIMsgFilterService.idl

Comment 1

[Ben Campbell]

Attachment: 1682945-remove-nsISimpleEnumerator-in-nsIMsgFilterService-1.patch

Comment 2

[Magnus Melin [:mkmelin]]

Comment on attachment 9197259 [details] [diff] [review]
1682945-remove-nsISimpleEnumerator-in-nsIMsgFilterService-1.patch

Review of attachment 9197259 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM, r=mkmelin

::: mailnews/search/public/nsIMsgFilterService.idl
@@ +53,5 @@
>  
>      /**
>       * get the list of custom actions
>       *
>       * @return   enumerator of nsIMsgFilterCustomAction objects

an array of

@@ +76,5 @@
>  
>      /**
>       * get the list of custom search terms
>       *
>       * @return   enumerator of nsIMsgSearchCustomTerm objects

an array of

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/f815431dbee5
Remove nsISimpleEnumerator use in nsIMsgFilterService. r=mkmelin

Comment 4

[Magnus Melin [:mkmelin]]

Noticed after the push, looks like you have the wrong email in your settings. Please fix.

Comment 5

[Ben Campbell]

(In reply to Magnus Melin [:mkmelin] from comment #4)

Noticed after the push, looks like you have the wrong email in your settings. Please fix.

Doh. Thanks for spotting that - fixed now!