Message242620
David I did the review and there's one thing that worries me the most, actually two:
1. changing the usual meaning of None in the IMAP's __init__ method, where None has the same meaning as True, where I think it should be the opposite.
2. I'm not sure we want to have UTF8 enabled based on the init's flag. I've seen our IMAP library as a wrapper around protocol itself. Whereas the user must be aware of required steps needed to proceed. In this case enabling UTF8 support is just the next command the client can, but doesn't have to sent directly, but only in AUTH state. |
|
Date |
User |
Action |
Args |
2015-05-05 21:17:36 | maciej.szulik | set | recipients:
+ maciej.szulik, barry, pitrou, eric.smith, r.david.murray, jesstess, BreamoreBoy, zvyn, Håkan Lövdahl |
2015-05-05 21:17:36 | maciej.szulik | set | messageid: <1430860656.01.0.716519804151.issue21800@psf.upfronthosting.co.za> |
2015-05-05 21:17:35 | maciej.szulik | link | issue21800 messages |
2015-05-05 21:17:35 | maciej.szulik | create | |
|