[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