homepage

The following code must display 3 instead of 0:
---
with open("x", "w") as f:
 f.write("xxx")
with open("x", "a") as f:
 print(f.tell())
---
The example works with Python 2.x, because file object is implemented 
using the FILE structure (fopen, ftell, etc.). fopen() "fixes" the 
offset if the file is opened in append mode, whereas open() doesn't do 
this for us :
---
import os
with open("x", "w") as f:
 f.write("xxx")
fd = os.open("x", os.O_RDONLY | os.O_APPEND)
print(os.lseek(fd, 0, 1))
---
display 0 instead of 3 on Python 2.x and 3.x.
It becomes a little bit more weird when you write something :-)
---
with open("x", "w") as f:
 f.write("xxx")
with open("x", "a") as f:
 f.write("y")
 print(f.tell())
---
displays... 4 (the correct position) on Python 2.x and 3.x.
I see (in GNU libc source code) that fopen() call lseek(fd, 0, 
SEEK_END) if the file is opened in append mode.

Patch _including a test_:
+	if (append)
+		lseek(self->fd, 0, SEEK_END);

Comments on the patch:
- you should check the error return of lseek() (and possibly wrap it in
Py_BEGIN/END_ALLOW_THREADS, see portable_lseek() in the same file)
- there should be a test for each of unbuffered IO (buffering=0),
buffered IO ("rb") and text IO ("r"). For text IO, the test shouldn't
test the actual value returned by tell(), only that it is > 0 (because
tell() in text mode is an opaque value and is not necessarily equal to a
byte position)

Patch version 2:
 - raise raise PyErr_SetFromErrno(PyExc_IOError) on lseek() error
 - add tests for unbuffered binary file and (buffered) text file
I use the type "long" to store the lseek() result, because I don't 
know if off_t is available on all OS. Py_off_t may be used, but it's 
defined above (after fileio_init). fileio_seekable() uses the 
type "int" for lseek() result, which looks worse than long :-)

> I use the type "long" to store the lseek() result, because I don't 
> know if off_t is available on all OS. Py_off_t may be used, but it's 
> defined above (after fileio_init).
Instead of checking the return type, you can first set errno to 0, and
then check errno after the function returns.
> fileio_seekable() uses the 
> type "int" for lseek() result, which looks worse than long :-)
Nice catch! http://bugs.python.org/issue5016
PS : about the patch, "0 < f.tell()" is really strange coding style...
"f.tell() > 0" looks much more consistent with the rest of the Python
code base

Last thing, in your patch there is a forward declaration to
portable_lseek but it doesn't look used anywhere...

New try (version 3):
 - reuse Py_off_t in fileio_init() instead of long
 - use Python coding style: f.tell() > 0
 - remove the unused forward declaration of portable_lseek()
This patch also prepares a fix for #5016.

Version 4: ok, let's use *portable*_lseek() instead of the "ugly" 
lseek() function (not compatible with large files on Windows).

Committed in r68835, r68836, r68837, r68838.