1412421 - Change some nsAuto[C]String parameters back to ns[C]String

Status: RESOLVED FIXED

(MailNews Core :: Backend, enhancement)

Description

[Jorg K (CEST = GMT+2)]

Let's fix some sins of the past ;-)

Comment 1

[Jorg K (CEST = GMT+2)]

Attachment: 1412421-minor-string-tweaks.patch

Eric, could you look at this for me. We're short in Mailnews reviewers. Thanks in advance.

Comment 2

[Eric Rahm [:erahm]]

Comment on attachment 8922956 [details] [diff] [review]
1412421-minor-string-tweaks.patch

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

Thanks for cleaning things up! Just a few minor tweaks suggested, r=me with those fixed.

::: mailnews/base/util/nsMsgUtils.cpp
@@ +2104,5 @@
>    return aValue;
>  }
>  
>  // Helper function to extract a query qualifier.
> +nsCString MsgExtractQueryPart(nsCString spec, const char* queryToExtract)

Please change `spec` to `const nsACString& spec` (it will avoid a copy).

::: mailnews/compose/src/nsMsgCompose.cpp
@@ +4205,5 @@
>   * This is intended to be used in for signature files, so that we can make sure
>   * images loaded into the editor are available on send.
>   */
>  nsresult
> +nsMsgCompose::ReplaceFileURLs(nsString &aData)

This can stay an auto string since it's an in/out param.

Comment 3

[Jorg K (CEST = GMT+2)]

Attachment: 1412421-minor-string-tweaks.patch (v2)

Fixed review issues, thanks for the quick turnaround.

I had to use
int32_t queryIndex = nsPromiseFlatCString(spec).Find(queryToExtract);
                     ^^^^^^^^^^^^^^^^^^^^^
since I can't .Find() on a nsACString. OK?

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/6982e252d9b9
Change some nsAuto[C]String parameters back to ns[C]String. r=erahm

Comment 5

[Eric Rahm [:erahm]]

(In reply to Jorg K (GMT+2) from comment #3)
> Created attachment 8922995 [details] [diff] [review]
> 1412421-minor-string-tweaks.patch (v2)
> 
> Fixed review issues, thanks for the quick turnaround.
> 
> I had to use
> int32_t queryIndex = nsPromiseFlatCString(spec).Find(queryToExtract);
>                      ^^^^^^^^^^^^^^^^^^^^^
> since I can't .Find() on a nsACString. OK?

Oh sorry! It has FindChar but not Find :( That works.