[PATCH 1/9] We had been setting this variable twice

Sebastian Spaeth Sebastian at SSpaeth.de
Tue Apr 26 11:31:32 BST 2011


This was in the code for a very long time, it seems.
Remove one instance, no functional changes.

Signed-off-by: Sebastian Spaeth <Sebastian at SSpaeth.de>
---
 offlineimap/folder/LocalStatus.py |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/offlineimap/folder/LocalStatus.py b/offlineimap/folder/LocalStatus.py
index 623829a..2bf46b8 100644
--- a/offlineimap/folder/LocalStatus.py
+++ b/offlineimap/folder/LocalStatus.py
@@ -29,7 +29,6 @@ class LocalStatusFolder(BaseFolder):
         self.sep = '.'
         self.config = config
         self.dofsync = config.getdefaultboolean("general", "fsync", True)
-        self.filename = os.path.join(root, name)
         self.filename = repository.getfolderfilename(name)
         self.messagelist = None
         self.repository = repository
@@ -119,6 +118,7 @@ class LocalStatusFolder(BaseFolder):
         if uid < 0:
             # We cannot assign a uid.
             return uid
+            #TODO: what to do in case uid==0?
 
         if uid in self.messagelist:     # already have it
             self.savemessageflags(uid, flags)
-- 
1.7.4.1





More information about the OfflineIMAP-project mailing list