Closed
Bug 239902
Opened 20 years ago
Closed 20 years ago
cleaning up imap inbox on exit with inbox configured for offline use leaves temp files
Categories
(MailNews Core :: Backend, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Bienvenu, Assigned: Bienvenu)
Details
Attachments
(1 file)
1.62 KB,
patch
|
mscott
:
superreview+
chofmann
:
approval1.7+
|
Details | Diff | Splinter Review |
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•20 years ago
|
||
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•20 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•20 years ago
|
Attachment #145631 -
Flags: superreview?(mscott) → superreview+
Assignee | ||
Updated•20 years ago
|
Attachment #145631 -
Flags: approval1.7?
Comment 3•20 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•20 years ago
|
||
er, 1.7
Assignee | ||
Comment 5•20 years ago
|
||
fixed.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Product: MailNews → Core
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
•