183560 - mail copy is very inefficient

Status: RESOLVED FIXED

(MailNews Core :: Backend, defect)

Description

[Colin Blake]

As reported in bug 182227, we need to do something about the way mailnews copies
a mail message (for example, when it copies it to the Sent folder). Here's what
its doing.

We read 10k bytes into a buffer, return just the first line, seek to the end of
the first line, then read 10k bytes again, return the second line, seek to the
end of the second line, etc. Talk about slow. No wonder I'm seeing reports of
disk grinding noises. I don't think you could come up with a slower way to copy
a file if you tried!!!

Comment 1

[Andrew Hagen]

*** Bug 182227 has been marked as a duplicate of this bug. ***

Comment 2

[Andrew Hagen]

Reporter, is this on an IMAP account?

Comment 3

[Colin Blake]

Nope, POP3.

Comment 4

[David :Bienvenu]

taking.

Comment 5

[David :Bienvenu]

Attachment: partial fix

this will speed up copying quite a bit, and fix the disk grinding. It doesn't
have to do with your diagnosis of the problem, Colin, but it gives a huge perf
win so I'm attaching it to this bug.

Comment 6

[David :Bienvenu]

Comment on attachment 121081 [details] [diff] [review]
partial fix

this has r/sr=sspitzer

Comment 7

[David :Bienvenu]

marking fixed