Open Bug 994313 Opened 11 years ago Updated 2 years ago

remove m_status member from nsFolderCompactState

Categories

(MailNews Core :: Database, defect)

Product:
MailNews Core
Component:
Database
Type:
defect

Tracking

(Not tracked)

People

(Reporter: aceman, Unassigned)

References

Details

+++ This bug was initially created as a clone of Bug #765471 comment 18 +++ neil@parkwaycc.co.uk 2014-04-09 22:50:46 CEST (In reply to aceman from comment #17) > (In reply to comment #16) > > OK, maybe I can persuade aceman or someone to remove all the references to > > m_status and call CleanupTempFilesAfterError() in a separate bug. > > If you file it and add some more context we may look at it :) What context do you need? nsFolderCompactState currently has a member variable m_status which is used to clean up the temp files after an error. However it only does this once the compactor is destroyed, when as this bug shows the temp files should be deleted immediately. This also applies to nsOfflineStoreCompactState but as that inherits m_status it will need to be fixed at the same time. This will then have the effect of porting this bug to cover offline store compaction.
Severity: normal → minor
Severity: minor → S4
You need to log in before you can comment on or make changes to this bug.