Remove dead variable "messages" from archiving code (BatchMessageMover)

RESOLVED FIXED in seamonkey2.1a2

Status

SeaMonkey
MailNews: General
--
trivial
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)

Tracking

Trunk
seamonkey2.1a2

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
Created attachment 450882 [details] [diff] [review]
remove dead variable [Checkin: comment 2]

Found while looking into porting some archiving enhancements from TB: The "messages" variable inside BatchMessageMover is unused. It was introduced with TB bug 451995 and copied blindly (my fault) in SM bug 482458. Time to get rid of it.
Attachment #450882 - Flags: superreview?(bienvenu)
Attachment #450882 - Flags: review?(neil)

Comment 1

7 years ago
Comment on attachment 450882 [details] [diff] [review]
remove dead variable [Checkin: comment 2]

thx, looks fine, rs=me.
Attachment #450882 - Flags: superreview?(bienvenu) → superreview+

Updated

7 years ago
Attachment #450882 - Flags: review?(neil) → review+
(Assignee)

Comment 2

7 years ago
Comment on attachment 450882 [details] [diff] [review]
remove dead variable [Checkin: comment 2]

http://hg.mozilla.org/comm-central/rev/bede1fff019d
Attachment #450882 - Attachment description: remove dead variable → remove dead variable [Checkin: comment 2]
(Assignee)

Updated

7 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1a2
You need to log in before you can comment on or make changes to this bug.