aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorZack Weinberg <zackw@panix.com>2018-03-07 14:31:58 -0500
committerGabriel F. T. Gomes <gabriel@inconstante.eti.br>2018-12-05 18:15:42 -0200
commit349718d4d7841df46bcc36df9bc2baef4c40d6f5 (patch)
tree1c6ae071a4a6d063e551ae6577b04d952262daea /ChangeLog
parentY2038: make __tz_convert compatible with 64-bit-time (diff)
downloadglibc-349718d4d7841df46bcc36df9bc2baef4c40d6f5.tar.gz
glibc-349718d4d7841df46bcc36df9bc2baef4c40d6f5.tar.bz2
glibc-349718d4d7841df46bcc36df9bc2baef4c40d6f5.zip
Add __vfscanf_internal and __vfwscanf_internal with flags arguments.
There are two flags currently defined: SCANF_LDBL_IS_DBL is the mode used by __nldbl_ scanf variants, and SCANF_ISOC99_A is the mode used by __isoc99_ scanf variants. In this patch, the new functions honor these flag bits if they're set, but they still also look at the corresponding bits of environmental state, and callers all pass zero. The new functions do *not* have the "errp" argument possessed by _IO_vfscanf and _IO_vfwscanf. All internal callers passed NULL for that argument. External callers could theoretically exist, so I preserved wrappers, but they are flagged as compat symbols and they don't preserve the three-way distinction among types of errors that was formerly exposed. These functions probably should have been in the list of deprecated _IO_ symbols in 2.27 NEWS -- they're not just aliases for vfscanf and vfwscanf. (It was necessary to introduce ldbl_compat_symbol for _IO_vfscanf. Please check that part of the patch very carefully, I am still not confident I understand all of the details of ldbl-opt.) This patch also introduces helper inlines in libio/strfile.h that encapsulate the process of initializing an _IO_strfile object for reading. This allows us to call __vfscanf_internal directly from sscanf, and __vfwscanf_internal directly from swscanf, without duplicating the initialization code. (Previously, they called their v-counterparts, but that won't work if we want to control *both* C99 mode and ldbl-is-dbl mode using the flags argument to__vfscanf_internal.) It's still a little awkward, especially for wide strfiles, but it's much better than what we had. Tested for powerpc and powerpc64le.
Diffstat (limited to 'ChangeLog')
-rw-r--r--ChangeLog67
1 files changed, 67 insertions, 0 deletions
diff --git a/ChangeLog b/ChangeLog
index 935863b7b2..d2a1888554 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,70 @@
+2018-12-05 Zack Weinberg <zackw@panix.com>
+ Gabriel F. T. Gomes <gabriel@inconstante.eti.br>
+
+ * libio/libioP.h (SCANF_LDBL_IS_DBL, SCANF_ISOC99_A): New constants.
+ (__vfscanf_internal, __vfwscanf_internal): New function prototypes.
+ * libio/libio.h: Remove libc_hidden_proto for _IO_vfscanf.
+ * libio/strfile.h: Add multiple inclusion guard.
+ (_IO_strfile_read, _IO_strfile_readw): New inline functions.
+
+ * sysdeps/generic/math_ldbl_opt.h: Include shlib-compat.h, for
+ consistency with the other version of this file.
+ (ldbl_compat_symbol): New macro.
+ * sysdeps/ieee754/ldbl-opt/math_ldbl_opt.h (ldbl_compat_symbol):
+ New macro.
+
+ * stdio-common/vfscanf-internal.c: Rename from vfscanf.c.
+ Define __vfscanf_internal or __vfwscanf_internal, depending on
+ COMPILE_WSCANF; don't define any other public symbols.
+ Remove errval and code to set errp.
+ Temporarily check __ldbl_is_dbl and _IO_FLAGS2_SCANF_STD as well
+ as the mode_flags argument.
+ (encode_error, conv_error, input_error): Don't set errval.
+ * stdio-common/vfwscanf-internal.c: Rename from vfwscanf.c.
+ Include vfscanf-internal.c.
+ * stdio-common/vfscanf.c: New file defining the public entry
+ point vfscanf, which calls __vfscanf_internal.
+ * stdio-common/vfwscanf.c: New file defining the public entry
+ point vfwscanf, which calls __vfwscanf_internal.
+
+ * stdio-common/iovfscanf.c: New file.
+ * stdio-common/iovfwscanf.c: Likewise.
+
+ * stdio-common/Makefile (routines): Add vfscanf-internal,
+ vfwscanf-internal, iovfscanf, iovfwscanf.
+ * stdio-common/Versions: Mention GLIBC_2.29, so that
+ it can be used in SHLIB_COMPAT expressions.
+ * sysdeps/ieee754/ldbl-opt/nldbl-compat.c (__nldbl__IO_vfscanf):
+ Wrap definition and compat_symbol line in #if SHLIB_COMPAT.
+ Call __vfscanf_internal, instead of _IO_vfscanf.
+ (__nldbl___vfscanf): Call __vfscanf_internal, instead of
+ _IO_vfscanf.
+ (__nldbl_vfwscanf): Call __vfwscanf_internal, instead of
+ _IO_vfwscanf.
+
+ * libio/iovsscanf.c: Clean up includes, when possible. Use
+ _IO_strfile_read or _IO_strfile_readw, when needed. Call
+ __vfscanf_internal or __vfwscanf_internal directly.
+ * libio/iovswscanf.c: Likewise.
+ * libio/swscanf.c: Likewise.
+ * libio/vscanf.c: Likewise.
+ * libio/vwscanf.c: Likewise.
+ * libio/wscanf.c: Likewise.
+ * stdio-common/isoc99_fscanf.c: Likewise.
+ * stdio-common/isoc99_scanf.c: Likewise.
+ * stdio-common/isoc99_sscanf.c: Likewise.
+ * stdio-common/isoc99_vfscanf.c: Likewise.
+ * stdio-common/isoc99_vscanf.c: Likewise.
+ * stdio-common/isoc99_vsscanf.c: Likewise.
+ * stdio-common/scanf.c: Likewise.
+ * stdio-common/sscanf.c: Likewise.
+ * wcsmbs/isoc99_fwscanf.c: Likewise.
+ * wcsmbs/isoc99_swscanf.c: Likewise.
+ * wcsmbs/isoc99_vfwscanf.c: Likewise.
+ * wcsmbs/isoc99_vswscanf.c: Likewise.
+ * wcsmbs/isoc99_vwscanf.c: Likewise.
+ * wcsmbs/isoc99_wscanf.c: Likewise.
+
2018-12-05 Albert ARIBAUD <albert.aribaud@3adev.fr>
* include/time.h