559801 - Don't use native charset <-> unicode charset if possible

Status: NEW

(MailNews Core :: Backend, defect)

Description

[Makoto Kato [:m_kato]]

There is the unnecessary conversion of native charset vs unicode charset at some lines.  This conversion may loose characters on Windows.

http://mxr.mozilla.org/comm-central/source/mailnews/local/src/nsLocalMailFolder.cpp#751
http://mxr.mozilla.org/comm-central/source/mailnews/local/src/nsLocalMailFolder.cpp#1151
http://mxr.mozilla.org/comm-central/source/mailnews/imap/src/nsImapMailFolder.cpp#565

Comment 1

[Wayne Mery (:wsmwk)]

(In reply to Makoto Kato from comment #0)
> There is the unnecessary conversion of native charset vs unicode charset at
> some lines.  This conversion may loose characters on Windows.
> 
> http://mxr.mozilla.org/comm-central/source/mailnews/local/src/
> nsLocalMailFolder.cpp#751
> http://mxr.mozilla.org/comm-central/source/mailnews/local/src/
> nsLocalMailFolder.cpp#1151
> http://mxr.mozilla.org/comm-central/source/mailnews/imap/src/
> nsImapMailFolder.cpp#565

Makoto-san, is this still a problem?  I'm not seeing (in current version) what source lines might do this

Comment 2

[Makoto Kato [:m_kato]]

mailnews/local and mailnews/imap are fixed, but some code such as folder compactor still needs.