Open Bug 1118132 Opened 9 years ago Updated 2 years ago

Refactor nsLocalMailFolder::CopyMessages so that message begin/end processing occurs in Start/EndMessage() methods

Categories

(MailNews Core :: Backend, defect)

Product:
MailNews Core
Component:
Backend
Platform:
x86_64
Windows 8.1
Type:
defect

Tracking

(Not tracked)

People

(Reporter: rkent, Unassigned)

References

Details

This bug was created as a followup to issues pointed out in bug 856519.

Copy processing in nsMsgLocalFolder is done using methods from nsICopyMessageListener, which has both begin/endCopy as well as start/endMessage methods. But in many cases the begin/endCopy methods are being used for operations that happen at the beginning and ending of individual messages, which results in duplication of code as well as regressions from the confusion.

The specific request of this bug is to rework this so that start/endMessage is used for all per-message processing, and begin/endCopy is used for operations at the beginning and ending of a sequence of possibly multiple messages.

At the same time, struct nsLocalMailCopyState should be much more thoroughly documented.
See Also: → 856519
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.