Mirror
/
cpython
mirror of https://github.com/python/cpython
4
1
Fork 0
Code Issues Releases Wiki Activity

_portable_fseek(): 

Subtlety on Windows: if we change test_largefile.py to use a file
> 4GB, it still fails.  A debug session suggests this is because
fseek(fp, 0, 2) refuses to seek to the end of the file when the file
is > 4GB, because it uses the SetFilePointer() in 32-bit mode.

But it only fails when we seek relative to the end of the file,
because in the other seek modes only calls to fgetpos() and fsetpos()
are made, which use Get/SetFilePointer() in 64-bit mode.  Solution:
#ifdef MS_WInDOWS, replace the call to fseek(fp, ...) with a call to
_lseeki64(fileno(fp), ...).  Make sure to call fflush(fp) first.

(XXX Could also replace the entire branch with a call to _lseeki64().
Would that be more efficient?  Certainly less generated code.)

(XXX This needs more testing.  I can't actually test that it works for
files >4GB on my Win98 machine, because the filesystem here won't let
me create files >=4GB at all.  Tim should test this on his Win2K
machine.)
This commit is contained in:
Guido van Rossum 2001-09-10 20:43:35 +00:00
parent c4c062f507
commit 8b4e43e768
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Objects/fileobject.c
View File

@ -239,8 +239,14 @@ _portable_fseek(FILE *fp, Py_off_t offset, int whence)
fpos_t pos; fpos_t pos;
switch (whence) { switch (whence) {
case SEEK_END: case SEEK_END:
#ifdef MS_WINDOWS
fflush(fp);
if (_lseeki64(fileno(fp), 0, 2) == -1)
return -1;
#else
if (fseek(fp, 0, SEEK_END) != 0) if (fseek(fp, 0, SEEK_END) != 0)
return -1; return -1;
#endif
/* fall through */ /* fall through */
case SEEK_CUR: case SEEK_CUR:
if (fgetpos(fp, &pos) != 0) if (fgetpos(fp, &pos) != 0)