Message 148846 - Python tracker

➜

This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author	pitrou
Recipients	Jon.Kuhn, orsenthil, pitrou, r.david.murray
Date	2011-12-04.09:20:47
SpamBayes Score	0.00031470894
Marked as misclassified	No
Message-id	<[email protected]>
In-reply-to

Content
Hello Jon, and thanks for the patch. I have a couple of comments: - readinto() shouldn't return None but 0 when there is nothing to read (this corresponds to read() returning b"") - I see _read_chunked() is only ever called with amt=None, so perhaps it can be simplified? Also, a nitpick: the doc entry needs a "versionadded" tag.

Hello Jon, and thanks for the patch. I have a couple of comments:

- readinto() shouldn't return None but 0 when there is nothing to read (this corresponds to read() returning b"")

- I see _read_chunked() is only ever called with amt=None, so perhaps it can be simplified?

Also, a nitpick: the doc entry needs a "versionadded" tag.

History
Date	User	Action	Args
2011-12-04 09:21:48	pitrou	set	recipients: + pitrou, orsenthil, r.david.murray, Jon.Kuhn
2011-12-04 09:21:48	pitrou	set	messageid: <[email protected]>
2011-12-04 09:20:47	pitrou	link	issue13464 messages
2011-12-04 09:20:47	pitrou	create