Closed Bug 2052827 Opened 19 days ago Closed 18 days ago

Fix incorrect use of nsString() and nsCString() to reduce unneeded memory churn and string copying.

Categories

(MailNews Core :: General, task)

Unspecified
All
task

Tracking

(Not tracked)

RESOLVED FIXED
154 Branch

People

(Reporter: yury_ivanovich, Assigned: yury_ivanovich)

Details

(Keywords: perf)

Attachments

(1 file)

Using nsString/nsCString instead of PromiseFlatString/PromiseFlatCString or nsDependentString/nsDependentCString causes unneeded memory churn and string copying.

I fixed some of it in https://hg-edge.mozilla.org/comm-central/rev/dfe6f47ad76e, but unfortunately bad string use has returned.

Patch to follow.

Assignee: nobody → Yury.Ivanovich
Status: NEW → ASSIGNED
Keywords: perf
OS: Unspecified → All
Summary: Fix incorrect use of nsString() and nsCString() → Fix incorrect use of nsString() and nsCString() to reduce unneeded memory churn and string copying.
Target Milestone: --- → 154 Branch

Pushed by kaie@kuix.de:
https://hg.mozilla.org/comm-central/rev/997f015a926a
Fix incorrect use of nsString() and nsCString(). r=BenC

Status: ASSIGNED → RESOLVED
Closed: 18 days ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: