STINNER Victor added the comment:

> Patch LGTM.

Thanks for the review.

> Here is a test for it.

Your test does not pass because Linux truncates the read() size to 2GB - 4096. 
I tested with /dev/zero device and with a regular file.

I wrote a simplified test to just check that size larger than INT_MAX does not 
emit an OverflowError.

----------
resolution:  -> fixed
status: open -> closed

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue21932>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to