Replace use of nsCString(xx).get() with PromiseFlatCString().get() in mailnews
Categories
(MailNews Core :: General, task)
Tracking
(thunderbird_esr91 fixed, thunderbird95 fixed)
People
(Reporter: rachel, Assigned: rachel)
References
Details
Attachments
(1 file)
(deleted),
patch
|
mkmelin
:
review+
wsmwk
:
approval-comm-beta+
wsmwk
:
approval-comm-esr91+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #1739784 +++
See bug 1739784 comment #17 and below.
Assignee | ||
Comment 1•3 years ago
|
||
We'll follow up with a patch to change some of the interfaces to reduce the conversions. For the
const nsPromiseFlatCString& flat = PromiseFlatCString(aSpec);
see M-C examples here:
https://searchfox.org/comm-central/search?q=nsPromiseFlatCString&path=netwerk&case=false®exp=false
Updated•3 years ago
|
Assignee | ||
Comment 2•3 years ago
|
||
Follow-up in bug 1741619, this is rippling out terribly.
Updated•3 years ago
|
Updated•3 years ago
|
Updated•3 years ago
|
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/f1a7bcd6d770
Replace use of nsCString().get() with PromiseFlatCString().get() in mailnews. r=mkmelin
Assignee | ||
Comment 4•3 years ago
|
||
Comment on attachment 9251108 [details] [diff] [review]
1741559-replace-nsCString-get.patch
[Approval Request Comment]
Regression caused by (bug #): bug 1571672
User impact if declined: Various things don't work on folders with non-ASCII names: Saves multiple drafts, virtual folders don't work, etc.
Testing completed (on c-c, etc.): Yes.
Risk to taking this patch (and alternatives if risky):
Not risky, mostly mechanical change of raw string API to "smart" string API allowing for non-ASCII strings to be correctly passed around.
Part of a pack of four bugs which need to be uplifted in this order:
Bug 1741559, bug 1741619, bug 1739814, bug 1739903. Bug 1739784 was the first one in the series and that's already on ESR.
Comment 5•3 years ago
|
||
Comment on attachment 9251108 [details] [diff] [review]
1741559-replace-nsCString-get.patch
[Triage Comment]
Approved for beta
Comment 6•3 years ago
|
||
bugherder uplift |
Thunderbird 95.0b5:
https://hg.mozilla.org/releases/comm-beta/rev/1a32d92a0836
Comment 7•3 years ago
|
||
Comment on attachment 9251108 [details] [diff] [review]
1741559-replace-nsCString-get.patch
[Triage Comment]
Approved for esr91
Comment 8•3 years ago
|
||
bugherder uplift |
Thunderbird 91.4.1:
https://hg.mozilla.org/releases/comm-esr91/rev/b4c26539b451
Description
•