]> code.delx.au - offlineimap/commitdiff
helpful comments
authorAdam Spiers <darcs@adamspiers.org>
Tue, 16 May 2006 03:40:23 +0000 (04:40 +0100)
committerAdam Spiers <darcs@adamspiers.org>
Tue, 16 May 2006 03:40:23 +0000 (04:40 +0100)
offlineimap/folder/IMAP.py

index 2768105dcf4c748b6498d1938fe6e9e5307f5dba..3a0ea1aa22f766882375a9124048b6775a0863a9 100644 (file)
@@ -234,6 +234,7 @@ class IMAPFolder(BaseFolder):
             ui.debug('imap', 'savemessage: first attempt to get new UID')
             uid = self.savemessage_searchforheader(imapobj, headername,
                                                    headervalue)
+            # See docs for savemessage in Base.py for explanation of this and other return values
             if uid <= 0:
                 ui.debug('imap', 'savemessage: first attempt to get new UID failed.  Going to run a NOOP and try again.')
                 assert(imapobj.noop()[0] == 'OK')
@@ -242,8 +243,9 @@ class IMAPFolder(BaseFolder):
         finally:
             self.imapserver.releaseconnection(imapobj)
 
-        if uid:
+        if uid: # avoid UID FETCH 0 crash happening later on
             self.messagelist[uid] = {'uid': uid, 'flags': flags}
+
         ui.debug('imap', 'savemessage: returning %d' % uid)
         return uid