Author martin.panter
Recipients martin.panter, ncoghlan, pitrou, serhiy.storchaka, vstinner
Date 2015-10-10.04:01:40
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1444449701.26.0.110707845792.issue25357@psf.upfronthosting.co.za>
In-reply-to
Content
The “newline” name seems as good as any. The patch looks good in general. I left a few comments.

I agree with not bothering for the UU encoder. That encoding has an embedded line length byte at the start of each line, so it makes less sense to omit newlines. Base-64 is used in places such as URLs where multiline output is not relevant, but I don’t know of a similar case for the UU encoding.

If I was doing it, I wouldn’t bother with a test case for keyword-only-ness, but suit yourself.
History
Date User Action Args
2015-10-10 04:01:41martin.pantersetrecipients: + martin.panter, ncoghlan, pitrou, vstinner, serhiy.storchaka
2015-10-10 04:01:41martin.pantersetmessageid: <1444449701.26.0.110707845792.issue25357@psf.upfronthosting.co.za>
2015-10-10 04:01:41martin.panterlinkissue25357 messages
2015-10-10 04:01:40martin.pantercreate