cleaning up imap inbox on exit with inbox configured for offline use leaves temp files

RESOLVED FIXED

Status

MailNews Core
Backend
RESOLVED FIXED
14 years ago
10 years ago

People

(Reporter: Bienvenu, Assigned: Bienvenu)

Tracking

Trunk
x86
Windows XP

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

14 years ago
If you set up your imap inbox for offline use, and configure the server to
expunge the inbox on exit, we'll leave temp files around because we try to
compact the offline inbox store on exit, but exit before the offline store
compact is finished. This causes us to leave temp files around in the imap
directory. The temporary fix is to not compact the offline store when the msg
window is null, and make it so the msg window is only null when doing a compact
at exit.
(Assignee)

Comment 1

14 years ago
Created attachment 145631 [details] [diff] [review]
proposed fix

the first part of the fix is to make sure m_window gets set in
nsMsgFolderCompactState so that when we do a compact all, the compact call for
folders after the first call get the msg window passed in. The second part is
to not compact the offline store when we don't have a msg window.

When we re-architect the shutdown code, this will all get cleaned up.
(Assignee)

Comment 2

14 years ago
Comment on attachment 145631 [details] [diff] [review]
proposed fix

I will try to get this into 1.7
Attachment #145631 - Flags: superreview?(mscott)

Updated

14 years ago
Attachment #145631 - Flags: superreview?(mscott) → superreview+
(Assignee)

Updated

14 years ago
Attachment #145631 - Flags: approval1.7?

Comment 3

14 years ago
Comment on attachment 145631 [details] [diff] [review]
proposed fix

a=chofmann for 1.6
Attachment #145631 - Flags: approval1.7? → approval1.7+

Comment 4

14 years ago
er, 1.7
(Assignee)

Comment 5

14 years ago
fixed.
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.