Message127097
> 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). |
|
Date |
User |
Action |
Args |
2011-01-26 10:00:29 | vstinner | set | recipients:
+ vstinner, akuchling, georg.brandl, rhettinger, holdenweb, pitrou, giampaolo.rodola, lregebro, r.david.murray, sdaoden |
2011-01-26 10:00:29 | vstinner | set | messageid: <1296036029.66.0.291715472733.issue9124@psf.upfronthosting.co.za> |
2011-01-26 10:00:29 | vstinner | link | issue9124 messages |
2011-01-26 10:00:29 | vstinner | create | |
|