diff options
| author | Siddhesh Poyarekar <siddhesh@redhat.com> | 2014-03-04 07:45:58 +0530 |
|---|---|---|
| committer | Siddhesh Poyarekar <siddhesh@redhat.com> | 2014-03-04 07:45:58 +0530 |
| commit | 000232b9bcbf194f1e5fd0ff380000f341505405 (patch) | |
| tree | 252e22a971caa7021ae130c080ccb161daba33cc /libio/Makefile | |
| parent | 116f4d1ad64a061488b29ef89f1a60f5e8c5963b (diff) | |
| download | glibc-000232b9bcbf194f1e5fd0ff380000f341505405.tar glibc-000232b9bcbf194f1e5fd0ff380000f341505405.tar.gz glibc-000232b9bcbf194f1e5fd0ff380000f341505405.tar.bz2 glibc-000232b9bcbf194f1e5fd0ff380000f341505405.zip | |
Separate ftell from fseek logic and avoid modifying FILE data (#16532)
ftell semantics are distinct from fseek(SEEK_CUR) especially when it
is called on a file handler that is not yet active. Due to this
caveat, much care needs to be taken while modifying the handler data
and hence, this first iteration on separating out ftell focusses on
maintaining handler data integrity at all times while it figures out
the current stream offset. The result is that it makes a syscall for
every offset request.
There is scope for optimizing this by caching offsets when we know
that the handler is active. A simple way to find out is when the
buffers have data. It is not so simple to find this out when the
buffer is empty without adding some kind of flag.
Diffstat (limited to 'libio/Makefile')
| -rw-r--r-- | libio/Makefile | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libio/Makefile b/libio/Makefile index 22dbcae0fd..488ee51f0e 100644 --- a/libio/Makefile +++ b/libio/Makefile @@ -60,7 +60,7 @@ tests = tst_swprintf tst_wprintf tst_swscanf tst_wscanf tst_getwc tst_putwc \ tst-wmemstream1 tst-wmemstream2 \ bug-memstream1 bug-wmemstream1 \ tst-setvbuf1 tst-popen1 tst-fgetwc bug-wsetpos tst-fseek \ - tst-fwrite-error tst-ftell-partial-wide + tst-fwrite-error tst-ftell-partial-wide tst-ftell-active-handler ifeq (yes,$(build-shared)) # Add test-fopenloc only if shared library is enabled since it depends on # shared localedata objects. |