Bug 234867 Turn some out params into retvals (not a functional change as all methods are noscript) r+sr=bienvenu
authorNeil Rashbrook <neil@parkwaycc.co.uk>
Wed, 17 Sep 2008 14:17:29 +0100
changeset 355 e8ba2f193d98d4ee8d39c37372c458db99202f97
parent 354 c1f31d315bfaa0e6240447cf7f93474b30dd695e
child 356 35e89cc8d7963f49df3dd074487a26d194cf047b
push idunknown
push userunknown
push dateunknown
bugs234867
Bug 234867 Turn some out params into retvals (not a functional change as all methods are noscript) r+sr=bienvenu
mailnews/mime/public/nsIMsgHeaderParser.idl
--- a/mailnews/mime/public/nsIMsgHeaderParser.idl
+++ b/mailnews/mime/public/nsIMsgHeaderParser.idl
@@ -86,59 +86,60 @@ interface nsIMsgHeaderParser : nsISuppor
 
 
   /* Given a string which contains a list of Header addresses, returns a
      comma-separated list of just the `user name' portions.  If any of
      the addresses doesn't have a name, then the mailbox is used instead.
 
      The caller may pass nsnull for charset and it will be interpreted as "us-ascii".
 
-     Caller must PL_strfree usernames.
+     Caller must NS_Free the return value.
   */
-  [noscript] void extractHeaderAddressNames (in string charset, in string line, out string userNames);
+  [noscript] string extractHeaderAddressNames (in string charset, in string line);
 
   /* Like MSG_ExtractHeaderAddressNames(), but only returns the first name
      in the list, if there is more than one. 
      
      The caller may pass nsnull for charset and it will be interpreted as "us-ascii".
 
-     Caller must call PL_strfree on name
+     Caller must NS_Free the return value.
   */
-  [noscript] void extractHeaderAddressName (in string charset, in string line, out string name);
+  [noscript] string extractHeaderAddressName (in string charset, in string line);
 
   /* Given a string which contains a list of Header addresses, returns a new
      string with the same data, but inserts missing commas, parses and reformats
      it, and wraps long lines with newline-tab.
 
      The caller may pass nsnull for charset and it will be interpreted as "us-ascii".
 
-     Caller must call PL_strfree on reformattedAddress
+     Caller must NS_Free the return value.
   */
-  [noscript] void reformatHeaderAddresses (in string charset, in string line, 
-                                out string reformattedAddress);
+  [noscript] string reformatHeaderAddresses (in string charset, in string line);
 
   /* Returns a copy of ADDRS which may have had some addresses removed.
      Addresses are removed if they are already in either ADDRS or OTHER_ADDRS.
      (If OTHER_ADDRS contain addresses which are not in ADDRS, they are not
      added.  That argument is for passing in addresses that were already
      mentioned in other header fields.)
 
      Addresses are considered to be the same if they contain the same mailbox
      part (case-insensitive.)  Real names and other comments are not compared.
 
      removeAliasesToMe allows the address parser to use the preference which
      contains regular expressions which also mean 'me' for the purpose of
      stripping the user's email address(es) out of addrs
 
      The caller may pass nsnull for charset and it will be interpreted as "us-ascii".
 
-     Caller must call PL_strfree on newAddress which is the return value.
+     Caller must NS_Free the return value.
   */
-  [noscript] void removeDuplicateAddresses (in string charset,  in string addrs, in string other_addrs, 
-                                 in PRBool removeAliasesToMe, out string newAddress);
+  [noscript] string removeDuplicateAddresses (in string charset,
+                                              in string addrs,
+                                              in string other_addrs,
+                                              in boolean removeAliasesToMe);
 
 
   /**
    * Given an e-mail address and a person's name, cons them together into a
    * single string, doing all the necessary quoting.
    *
    * @param  aName     The name of the sender.
    * @param  aAddress  The address of the sender.