Author vstinner
Recipients akuchling, georg.brandl, giampaolo.rodola, holdenweb, lregebro, pitrou, r.david.murray, rhettinger, sdaoden, vstinner
Date 2011-01-26.10:00:29
SpamBayes Score 5.81727e-05
Marked as misclassified No
Message-id <1296036029.66.0.291715472733.issue9124@psf.upfronthosting.co.za>
In-reply-to
Content
> I reverted r88197 because it was incorrect and caused an email test
> to fail. Once I come up with a test for it I'll fix it correctly.

test_mailbox is a good (indirect) test suite for this change. The problem of r88197 is that it replaces msg._payload by msg.get_payload() which is wrong. New attached patch mailbox.patch	keeps msg._payload unchanged, but don't call _has_surrogates() (do nothing) if msg._payload is None. This patch has no test, I'm unable to write a test for this case (directly with the email API).
History
Date User Action Args
2011-01-26 10:00:29vstinnersetrecipients: + vstinner, akuchling, georg.brandl, rhettinger, holdenweb, pitrou, giampaolo.rodola, lregebro, r.david.murray, sdaoden
2011-01-26 10:00:29vstinnersetmessageid: <1296036029.66.0.291715472733.issue9124@psf.upfronthosting.co.za>
2011-01-26 10:00:29vstinnerlinkissue9124 messages
2011-01-26 10:00:29vstinnercreate