Message161231
| Author |
r.david.murray |
| Recipients |
barry, eric.araujo, hynek, r.david.murray |
| Date |
2012年05月20日.20:18:51 |
| SpamBayes Score |
-1.0 |
| Marked as misclassified |
Yes |
| Message-id |
<1337545131.84.0.525710320917.issue14731@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
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. |
|
History
|
|---|
| 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 |
|