[Checkins] SVN: zope.sendmail/trunk/src/zope/sendmail/delivery.py - wrap a long line

Benji York benji at zope.com
Tue Oct 23 14:37:31 EDT 2007


Log message for revision 80970:
  - wrap a long line
  - enhance a comment
  

Changed:
  U   zope.sendmail/trunk/src/zope/sendmail/delivery.py

-=-
Modified: zope.sendmail/trunk/src/zope/sendmail/delivery.py
===================================================================
--- zope.sendmail/trunk/src/zope/sendmail/delivery.py	2007-10-23 18:14:23 UTC (rev 80969)
+++ zope.sendmail/trunk/src/zope/sendmail/delivery.py	2007-10-23 18:37:30 UTC (rev 80970)
@@ -289,7 +289,10 @@
                     if age is not None:
                         try:
                             if age > MAX_SEND_TIME:
-                                # the tmp file is "too old" remove it
+                                # the tmp file is "too old"; this suggests
+                                # that during an attemt to send it, the
+                                # process died; remove the tmp file so we
+                                # can try again
                                 os.unlink(tmp_filename)
                             else:
                                 # the tmp file is "new", so someone else may
@@ -299,9 +302,9 @@
                             # old, so it was unlinked
                         except OSError, e:
                             if e.errno == 2: # file does not exist
-                                # it looks like someone else removed the tmp file,
-                                # that's fine, we'll try to deliver the message
-                                # again later
+                                # it looks like someone else removed the tmp
+                                # file, that's fine, we'll try to deliver the
+                                # message again later
                                 continue
 
                     # now we know that the tmp file doesn't exist, we need to



More information about the Checkins mailing list