[go: nahoru, domu]

Open Bug 116442 Opened 23 years ago Updated 2 years ago

Inefficient "Save As" charset conversion for plain text mail if the charset is stateful

Categories

(MailNews Core :: Internationalization, defect)

defect

Tracking

(Not tracked)

People

(Reporter: nhottanscp, Assigned: smontagu)

References

Details

(Keywords: perf)

http://lxr.mozilla.org/seamonkey/source/mailnews/mime/src/mimetpla.cpp#449

ScanTXT takes unicode and there are two conversions (to unicode, and from
unicode) in case the charset is stateful (ISO-2022-JP). 
This is inefficient, the conversions are done per line, need to improve this part.
Blocks: 115288
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.9
Found that this is only used for SaveAs which is not performance critical.

440 if (obj->options->format_out != nsMimeOutput::nsMimeMessageSaveAs ||
441 !mailCharset || !nsMsgI18Nstateful_charset(mailCharset))


Target Milestone: mozilla0.9.9 → mozilla1.2
wontfix?
There is a way to improve by using the cached unicode converters in the MIME
object but I don't plan to do that very soon.
Target Milestone: mozilla1.2alpha → ---
Product: MailNews → Core
(In reply to comment #3)
> There is a way to improve by using the cached unicode converters in the MIME
> object but I don't plan to do that very soon.

Any update on this?
(In reply to comment #4)
> (In reply to comment #3)
> > There is a way to improve by using the cached unicode converters in the MIME
> > object but I don't plan to do that very soon.
> 
> Any update on this?

nhottanscp isn't about anymore, and I don't think anyone is watching or caring for the majority i18n bugs
Assignee: nhottanscp → smontagu
Status: ASSIGNED → NEW
Keywords: perf
QA Contact: ji → mailnews.i18n
Summary: Inefficient charset conversion for plain text mail if the charset is stateful → Inefficient "Save As" charset conversion for plain text mail if the charset is stateful
Product: Core → MailNews Core
Severity: normal → minor
Severity: minor → S4
You need to log in before you can comment on or make changes to this bug.