Closed Bug 1741559 Opened 3 years ago Closed 3 years ago

Replace use of nsCString(xx).get() with PromiseFlatCString().get() in mailnews

Categories

(MailNews Core :: General, task)

Tracking

(thunderbird_esr91 fixed, thunderbird95 fixed)

RESOLVED FIXED
96 Branch
Tracking Status
thunderbird_esr91 --- fixed
thunderbird95 --- fixed

People

(Reporter: rachel, Assigned: rachel)

References

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1739784 +++

See bug 1739784 comment #17 and below.

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&regexp=false

Attachment #9251108 - Flags: review?(standard8)
Attachment #9251108 - Flags: review?(standard8) → review?(mkmelin+mozilla)

Follow-up in bug 1741619, this is rippling out terribly.

Blocks: 1741619
Attachment #9251108 - Flags: review?(mkmelin+mozilla) → review+
Assignee: nobody → rachel
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Target Milestone: --- → 96 Branch

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

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED

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.

Attachment #9251108 - Flags: approval-comm-esr91?
Attachment #9251108 - Flags: approval-comm-beta?

Comment on attachment 9251108 [details] [diff] [review]
1741559-replace-nsCString-get.patch

[Triage Comment]
Approved for beta

Attachment #9251108 - Flags: approval-comm-beta? → approval-comm-beta+

Comment on attachment 9251108 [details] [diff] [review]
1741559-replace-nsCString-get.patch

[Triage Comment]
Approved for esr91

Attachment #9251108 - Flags: approval-comm-esr91? → approval-comm-esr91+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: