]> sourceware.org Git - glibc.git/commit - ChangeLog
Use cached offset in ftell when reliable
authorSiddhesh Poyarekar <siddhesh@redhat.com>
Tue, 4 Mar 2014 06:53:27 +0000 (12:23 +0530)
committerSiddhesh Poyarekar <siddhesh@redhat.com>
Tue, 4 Mar 2014 06:53:28 +0000 (12:23 +0530)
commitfa3cd24827d34a49e0a3a5cac56abbf8df74d8ac
tree67c2695a5d5b2a603d586339df17519a24fd17ee
parent000232b9bcbf194f1e5fd0ff380000f341505405
Use cached offset in ftell when reliable

The cached offset is reliable to use in ftell when the stream handle
is active.  We can consider a stream as being active when there is
unflushed data.  However, even in this case, we can use the cached
offset only when the stream is not being written to in a+ mode,
because this case may have unflushed data and a stale offset; the
previous read could have sent it off somewhere other than the end of
the file.

There were a couple of adjustments necessary to get this to work.
Firstly, fdopen now ceases to use _IO_attach_fd because it sets the
offset cache to the current file position.  This is not correct
because there could be changes to the file descriptor before the
stream handle is activated, which would not get reflected.

A similar offset caching action is done in _IO_fwide, claiming that
wide streams have 'problems' with the file offsets.  There don't seem
to be any obvious problems with not having the offset cache available,
other than that it will have to be queried in a subsequent
read/write/seek.  I have removed this as well.

The testsuite passes successfully with these changes on x86_64.
ChangeLog
libio/fileops.c
libio/iofdopen.c
libio/iofwide.c
libio/wfileops.c
This page took 0.04239 seconds and 5 git commands to generate.