Roundup Robot added the comment:
New changeset 9aba9ad6c15b by Victor Stinner in branch '3.2':
Issue #16367: Fix FileIO.readall() on Windows for files larger than 2 GB
http://hg.python.org/cpython/rev/9aba9ad6c15b
New changeset 5f96e4619ceb by Victor Stinner in branch '3.3':
(Merge 3.2) Issue
STINNER Victor added the comment:
Perhaps we could add a bigmem test for that (in test_fileio)?
Well, I'm too lazy to write such test. Instead of leaving the issue open for
weeks, I prefer to commit my patch with no new test (sorry!).
--
resolution: - fixed
status: open - closed
Antoine Pitrou added the comment:
Looks ok to me. Perhaps we could add a bigmem test for that (in test_fileio)?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16367
___
New submission from STINNER Victor:
The changeset 374dc910db33 fixed FileIO.readinto(), FileIO.write() and
os.write() on Windows for issues #9015, #9611, but FileIO.readall() was not
fixed and it has a similar bug.
It uses int n; to store the result of read(), so FileIO.readall() may also