Closed
Bug 67270
Opened 24 years ago
Closed 17 years ago
code cleanup and more performance for import mail
Categories
(MailNews Core :: Profile Migration, defect)
MailNews Core
Profile Migration
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: m_kato, Assigned: cavin)
References
Details
(Keywords: perf)
Attachments
(1 file)
10.91 KB,
patch
|
Details | Diff | Splinter Review |
code cleanup and more performance for import mail.
Reporter | ||
Comment 1•24 years ago
|
||
Reporter | ||
Comment 2•24 years ago
|
||
Do you know who can review Import compomnent???
Comment 3•24 years ago
|
||
tony would be the one to review this.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 5•24 years ago
|
||
makoto, don't do
nsString uniqueName;
do
nsAutoString uniqueName;
nsString and nsCString, as they allocate on the heap. nsAutoString and
nsCAutoString are on the stack.
also, instead of
NS_ConvertASCIItoUCS2("foo").GetUnicode()
do
NS_LITERAL_STRING("foo").Get()
"Imported Mail" should be moved out into a string bundle, instead of hard coded.
feel free to take a crack at that here, or I can open a new bug.
Comment 7•23 years ago
|
||
*** Bug 107985 has been marked as a duplicate of this bug. ***
Any chance some of the ideas/code in this patch can be brought to life again?
Blocks: 63759
QA Contact: gbush → stephend
Keywords: nsbeta1
Comment 9•23 years ago
|
||
reassigning to cavin. marking nsbeta1-.
Updated•20 years ago
|
Product: MailNews → Core
Comment 10•17 years ago
|
||
->WFM, unowned old bug with no usefulness anymore
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → WORKSFORME
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•