[Patches] [Patch #103476] Two Lib/mailbox.py fixes

noreply@sourceforge.net noreply@sourceforge.net
Wed, 31 Jan 2001 14:14:05 -0800


Patch #103476 has been updated. 

Project: python
Category: library
Status: Closed
Submitted by: bwarsaw
Assigned to : bwarsaw
Summary: Two Lib/mailbox.py fixes

Follow-Ups:

Date: 2001-Jan-31 14:14
By: bwarsaw

Comment:
Applying for Python 2.1a2.
-------------------------------------------------------

Date: 2001-Jan-31 13:58
By: bwarsaw

Comment:
I changed my mind on this.  For better backwards compatibility, I'm going
to leave UnixMailbox doing the strict checking, and add a
PortableUnixMailbox class which does the more general From_ line check.
-------------------------------------------------------

Date: 2001-Jan-31 13:27
By: bwarsaw

Comment:
Good idea about the StrictUnixMailbox subclass.  I'll add this, update NEWS
and check it in.
-------------------------------------------------------

Date: 2001-Jan-28 23:37
By: twouters

Comment:
Looks okay to me. I'm not sure if I'm "fine" with the idea of matching
from_lines without a preceding blank line, but I also don't want to invest
the time to rewrite the module to even allow that, so I can't expect anyone
else to :)

From-lineiness and the quoting of such differs greatly between mailers :P I
*think* this will work for 99.9% of the mailboxes out there (since I know
it works for Pine, Mutt, Nutscape and Eudorka (which uses an aditional file
for message state, though)) but I'm still a bit wary of forcing people in
the last promille to subclass and fiddle with internals themselves. Why not
provide a StrictMailbox class that does it for them ?

Don't forget Misc/NEWS, Barry. If I was supposed to mark it Accepted,
consider it marked Accepted :)

-------------------------------------------------------

Date: 2001-Jan-28 21:48
By: bwarsaw

Comment:
Assigned to Thomas for sanity checking.
-------------------------------------------------------

-------------------------------------------------------
For more info, visit:

http://sourceforge.net/patch/?func=detailpatch&patch_id=103476&group_id=5470