Message161231
I checked the indent fix into the wrong branch in my repo. It's in the other patch. I could fix that if desired.
The reason for making the base class public is to allow subclassing. Perhaps that is not needed for Compat32, though.
I don't think that the policy argument obsoletes the existing method options. It is easier to write:
gen.flatten(msg, linesep='\r\n')
than it is to write
gen.flatten(msg, policy=msg.policy.clone(linesep='\r\n'))
That is, when you want to make a change in just one method call, as opposed to globally, then the existing arguments are more convenient.
That said, a section in What's New describing when it might be a good idea to move to a global policy setting vs using the existing arguments may be worthwhile. |
|
Date |
User |
Action |
Args |
2012-05-20 20:18:51 | r.david.murray | set | recipients:
+ r.david.murray, barry, eric.araujo, hynek |
2012-05-20 20:18:51 | r.david.murray | set | messageid: <1337545131.84.0.525710320917.issue14731@psf.upfronthosting.co.za> |
2012-05-20 20:18:51 | r.david.murray | link | issue14731 messages |
2012-05-20 20:18:51 | r.david.murray | create | |
|