Last Comment Bug 571713 - Remove dead variable "messages" from archiving code (BatchMessageMover)
: Remove dead variable "messages" from archiving code (BatchMessageMover)
Status: RESOLVED FIXED
:
Product: SeaMonkey
Classification: Client Software
Component: MailNews: General (show other bugs)
: Trunk
: All All
: -- trivial (vote)
: seamonkey2.1a2
Assigned To: Jens Hatlak (:InvisibleSmiley)
:
Mentors:
Depends on: 482458
Blocks:
  Show dependency treegraph
 
Reported: 2010-06-12 14:52 PDT by Jens Hatlak (:InvisibleSmiley)
Modified: 2010-06-13 03:38 PDT (History)
0 users
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
remove dead variable [Checkin: comment 2] (1.99 KB, patch)
2010-06-12 14:52 PDT, Jens Hatlak (:InvisibleSmiley)
neil: review+
mozilla: superreview+
Details | Diff | Review

Description Jens Hatlak (:InvisibleSmiley) 2010-06-12 14:52:38 PDT
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.
Comment 1 David :Bienvenu 2010-06-12 15:48:51 PDT
Comment on attachment 450882 [details] [diff] [review]
remove dead variable [Checkin: comment 2]

thx, looks fine, rs=me.
Comment 2 Jens Hatlak (:InvisibleSmiley) 2010-06-13 03:38:01 PDT
Comment on attachment 450882 [details] [diff] [review]
remove dead variable [Checkin: comment 2]

http://hg.mozilla.org/comm-central/rev/bede1fff019d

Note You need to log in before you can comment on or make changes to this bug.