From patchwork Wed Jun 26 17:50:20 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Zack Weinberg X-Patchwork-Id: 33430 Received: (qmail 12479 invoked by alias); 26 Jun 2019 17:50:41 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 12352 invoked by uid 89); 26 Jun 2019 17:50:40 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-15.8 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, KAM_SHORT, KAM_STOCKGEN, SPF_HELO_PASS, SPF_PASS autolearn=ham version=3.3.1 spammy= X-HELO: l2mail1.panix.com From: Zack Weinberg To: libc-alpha@sourceware.org Cc: joseph@codesourcery.com, carlos@redhat.com Subject: [PATCH 16/25] Limit the set of strings.h functions also exposed in string.h. Date: Wed, 26 Jun 2019 13:50:20 -0400 Message-Id: <20190626175029.4699-7-zackw@panix.com> In-Reply-To: <20190626175029.4699-1-zackw@panix.com> References: <20190626175029.4699-1-zackw@panix.com> MIME-Version: 1.0 As someone who can remember when you might not be able to include both string.h and strings.h at the same time, it annoys me that strings.h still exists and is the only standard source for str(n)casecmp(_l) and ffs. I think it’s right that we expose those functions from string.h. However, there’s no reason we need to keep exposing the other obsolete functions that strings.h declares from string.h. This patch creates , which declares the non-obsolete functions whose official home is strings.h. strings.h includes it unconditionally, and string.h includes it under __USE_MISC, instead of strings.h. Two tests of the obsolete strings.h functions had to be adjusted. * string/strings.h (strcasecmp, strncasecmp, strcasecmp_l) (strncasecmp_l, ffs, ffsl, ffsll): Move declarations to... * string/bits/strings_x2k8.h: ... this new file. * string/Makefile: Install bits/strings_x2k8.h. * include/bits/strings_x2k8h: New wrapper. * string/string.h: Include bits/strings_x2k8.h instead of strings.h. * debug/tst-chk1.c, string/test-string.h: Include strings.h. * scripts/check-obsolete-constructs.py: string.h is no longer expected to include strings.h. * sysdeps/i386/i686/multiarch/bcopy.c * sysdeps/i386/i686/multiarch/bzero.c * sysdeps/i386/i686/multiarch/ifunc-impl-list.c * sysdeps/powerpc/powerpc32/power4/multiarch/ifunc-impl-list.c * sysdeps/powerpc/powerpc64/multiarch/bcopy-ppc64.c * sysdeps/powerpc/powerpc64/multiarch/bcopy.c * sysdeps/powerpc/powerpc64/multiarch/ifunc-impl-list.c * sysdeps/s390/bzero.c * sysdeps/s390/multiarch/ifunc-impl-list.c * sysdeps/sparc/sparc64/multiarch/bzero.c * sysdeps/sparc/sparc64/multiarch/ifunc-impl-list.c: Include strings.h. --- NEWS | 3 + debug/tst-chk1.c | 1 + include/bits/strings_x2k8.h | 1 + scripts/check-obsolete-constructs.py | 1 - string/Makefile | 2 +- string/bits/strings_x2k8.h | 70 +++++++++++++++++++ string/string.h | 2 +- string/strings.h | 39 +---------- string/test-string.h | 1 + sysdeps/i386/i686/multiarch/bcopy.c | 1 + sysdeps/i386/i686/multiarch/bzero.c | 1 + sysdeps/i386/i686/multiarch/ifunc-impl-list.c | 1 + .../power4/multiarch/ifunc-impl-list.c | 1 + .../powerpc/powerpc64/multiarch/bcopy-ppc64.c | 1 + sysdeps/powerpc/powerpc64/multiarch/bcopy.c | 1 + .../powerpc64/multiarch/ifunc-impl-list.c | 1 + sysdeps/s390/bzero.c | 1 + sysdeps/s390/multiarch/ifunc-impl-list.c | 1 + sysdeps/sparc/sparc64/multiarch/bzero.c | 1 + .../sparc/sparc64/multiarch/ifunc-impl-list.c | 1 + 20 files changed, 91 insertions(+), 40 deletions(-) create mode 100644 include/bits/strings_x2k8.h create mode 100644 string/bits/strings_x2k8.h diff --git a/NEWS b/NEWS index edf7739293..974ed39e4d 100644 --- a/NEWS +++ b/NEWS @@ -45,6 +45,9 @@ Deprecated and removed features, and other changes affecting compatibility: * The obsolete and never-implemented XSI STREAMS header files and have been removed. +* The obsolete functions bcmp, bcopy, bzero, index, and rindex are no + longer declared in , only . + * The typedefs u_int8_t, u_int16_t, u_int32_t, u_int64_t, and register_t are no longer defined by in strict conformance modes. These types were historically provided by on BSD systems, diff --git a/debug/tst-chk1.c b/debug/tst-chk1.c index c6f11af1ff..014b08fab7 100644 --- a/debug/tst-chk1.c +++ b/debug/tst-chk1.c @@ -30,6 +30,7 @@ #include #include #include +#include #include #include #include diff --git a/include/bits/strings_x2k8.h b/include/bits/strings_x2k8.h new file mode 100644 index 0000000000..7eddd92d93 --- /dev/null +++ b/include/bits/strings_x2k8.h @@ -0,0 +1 @@ +#include diff --git a/scripts/check-obsolete-constructs.py b/scripts/check-obsolete-constructs.py index e95e587d3e..bd5a97f76b 100755 --- a/scripts/check-obsolete-constructs.py +++ b/scripts/check-obsolete-constructs.py @@ -506,7 +506,6 @@ HEADER_ALLOWED_INCLUDES = { "inttypes.h": [ "stdint.h" ], "signal.h": [ "sys/ucontext.h" ], "stdlib.h": [ "alloca.h", "sys/types.h" ], - "string.h": [ "strings.h" ], "tgmath.h": [ "complex.h", "math.h" ], "threads.h": [ "time.h" ], diff --git a/string/Makefile b/string/Makefile index 3e5721e0f4..c9ebede768 100644 --- a/string/Makefile +++ b/string/Makefile @@ -23,7 +23,7 @@ subdir := string include ../Makeconfig headers := string.h bits/string_fortified.h \ - strings.h bits/strings_fortified.h \ + strings.h bits/strings_x2k8.h bits/strings_fortified.h \ byteswap.h bits/byteswap.h \ endian.h bits/endian.h bits/endianness.h \ bits/uintn-identity.h \ diff --git a/string/bits/strings_x2k8.h b/string/bits/strings_x2k8.h new file mode 100644 index 0000000000..dbcb0b2c31 --- /dev/null +++ b/string/bits/strings_x2k8.h @@ -0,0 +1,70 @@ +/* Copyright (C) 2019 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + . */ + +#ifndef _BITS_STRINGS_X2K8_H +#define _BITS_STRINGS_X2K8_H 1 + +/* This header file declares all of the strings.h functions that are + not marked as "obsolete" in POSIX.1-2008. As a GNU extension, + these functions are also made available via string.h. */ + +#if !defined _STRING_H && !defined _STRINGS_H +# error "Never include directly, use ." +#endif + +__BEGIN_DECLS + +/* Compare S1 and S2, ignoring case. */ +extern int strcasecmp (const char *__s1, const char *__s2) + __THROW __attribute_pure__ __nonnull ((1, 2)); + +/* Compare no more than N chars of S1 and S2, ignoring case. */ +extern int strncasecmp (const char *__s1, const char *__s2, size_t __n) + __THROW __attribute_pure__ __nonnull ((1, 2)); + +#ifdef __USE_XOPEN2K8 +/* POSIX.1-2008 extended locale interface (see locale.h). */ +# include + +/* Compare S1 and S2, ignoring case, using collation rules from LOC. */ +extern int strcasecmp_l (const char *__s1, const char *__s2, locale_t __loc) + __THROW __attribute_pure__ __nonnull ((1, 2, 3)); + +/* Compare no more than N chars of S1 and S2, ignoring case, using + collation rules from LOC. */ +extern int strncasecmp_l (const char *__s1, const char *__s2, + size_t __n, locale_t __loc) + __THROW __attribute_pure__ __nonnull ((1, 2, 4)); +#endif + +#if defined __USE_MISC || !defined __USE_XOPEN2K8 || defined __USE_XOPEN2K8XSI +/* Return the position of the first bit set in I, or 0 if none are set. + The least-significant bit is position 1, the most-significant 32. */ +extern int ffs (int __i) __THROW __attribute_const__; +#endif + +/* The following two functions are non-standard but necessary for non-32 bit + platforms. */ +#ifdef __USE_MISC +extern int ffsl (long int __l) __THROW __attribute_const__; +__extension__ extern int ffsll (long long int __ll) + __THROW __attribute_const__; +#endif + +__END_DECLS + +#endif /* bits/strings_x2k8.h */ diff --git a/string/string.h b/string/string.h index 2cc4f50090..6c9cdc0780 100644 --- a/string/string.h +++ b/string/string.h @@ -426,7 +426,7 @@ extern char *strerror_l (int __errnum, locale_t __l) __THROW; #endif #ifdef __USE_MISC -# include +# include /* Set N bytes of S to 0. The compiler will not delete a call to this function, even if S is dead after the call. */ diff --git a/string/strings.h b/string/strings.h index 17e1bf737f..bec458f1d7 100644 --- a/string/strings.h +++ b/string/strings.h @@ -21,6 +21,8 @@ #include #include +#include + /* Tell the caller that we provide correct C++ prototypes. */ #if defined __cplusplus && __GNUC_PREREQ (4, 4) # define __CORRECT_ISO_CPP_STRINGS_H_PROTO @@ -97,43 +99,6 @@ extern char *rindex (const char *__s, int __c) # endif #endif -#if defined __USE_MISC || !defined __USE_XOPEN2K8 || defined __USE_XOPEN2K8XSI -/* Return the position of the first bit set in I, or 0 if none are set. - The least-significant bit is position 1, the most-significant 32. */ -extern int ffs (int __i) __THROW __attribute_const__; -#endif - -/* The following two functions are non-standard but necessary for non-32 bit - platforms. */ -# ifdef __USE_MISC -extern int ffsl (long int __l) __THROW __attribute_const__; -__extension__ extern int ffsll (long long int __ll) - __THROW __attribute_const__; -# endif - -/* Compare S1 and S2, ignoring case. */ -extern int strcasecmp (const char *__s1, const char *__s2) - __THROW __attribute_pure__ __nonnull ((1, 2)); - -/* Compare no more than N chars of S1 and S2, ignoring case. */ -extern int strncasecmp (const char *__s1, const char *__s2, size_t __n) - __THROW __attribute_pure__ __nonnull ((1, 2)); - -#ifdef __USE_XOPEN2K8 -/* POSIX.1-2008 extended locale interface (see locale.h). */ -# include - -/* Compare S1 and S2, ignoring case, using collation rules from LOC. */ -extern int strcasecmp_l (const char *__s1, const char *__s2, locale_t __loc) - __THROW __attribute_pure__ __nonnull ((1, 2, 3)); - -/* Compare no more than N chars of S1 and S2, ignoring case, using - collation rules from LOC. */ -extern int strncasecmp_l (const char *__s1, const char *__s2, - size_t __n, locale_t __loc) - __THROW __attribute_pure__ __nonnull ((1, 2, 4)); -#endif - __END_DECLS #if __GNUC_PREREQ (3,4) && __USE_FORTIFY_LEVEL > 0 \ diff --git a/string/test-string.h b/string/test-string.h index befd3e9931..2c1c01699c 100644 --- a/string/test-string.h +++ b/string/test-string.h @@ -55,6 +55,7 @@ extern impl_t __start_impls[], __stop_impls[]; #include #include #include +#include #include #include #include diff --git a/sysdeps/i386/i686/multiarch/bcopy.c b/sysdeps/i386/i686/multiarch/bcopy.c index c2d1d25710..e1cd57c6e9 100644 --- a/sysdeps/i386/i686/multiarch/bcopy.c +++ b/sysdeps/i386/i686/multiarch/bcopy.c @@ -21,6 +21,7 @@ #if IS_IN (libc) # define bcopy __redirect_bcopy # include +# include # undef bcopy # define SYMBOL_NAME bcopy diff --git a/sysdeps/i386/i686/multiarch/bzero.c b/sysdeps/i386/i686/multiarch/bzero.c index 6b4834e9de..182a496af6 100644 --- a/sysdeps/i386/i686/multiarch/bzero.c +++ b/sysdeps/i386/i686/multiarch/bzero.c @@ -21,6 +21,7 @@ #if IS_IN (libc) # define bzero __redirect_bzero # include +# include # undef bzero # define SYMBOL_NAME bzero diff --git a/sysdeps/i386/i686/multiarch/ifunc-impl-list.c b/sysdeps/i386/i686/multiarch/ifunc-impl-list.c index e2428e0b11..5408aec908 100644 --- a/sysdeps/i386/i686/multiarch/ifunc-impl-list.c +++ b/sysdeps/i386/i686/multiarch/ifunc-impl-list.c @@ -18,6 +18,7 @@ #include #include +#include #include #include #include "init-arch.h" diff --git a/sysdeps/powerpc/powerpc32/power4/multiarch/ifunc-impl-list.c b/sysdeps/powerpc/powerpc32/power4/multiarch/ifunc-impl-list.c index 16a64beb33..ac6784a61b 100644 --- a/sysdeps/powerpc/powerpc32/power4/multiarch/ifunc-impl-list.c +++ b/sysdeps/powerpc/powerpc32/power4/multiarch/ifunc-impl-list.c @@ -18,6 +18,7 @@ #include #include +#include #include #include #include diff --git a/sysdeps/powerpc/powerpc64/multiarch/bcopy-ppc64.c b/sysdeps/powerpc/powerpc64/multiarch/bcopy-ppc64.c index 6772e0a867..9ba96f7d45 100644 --- a/sysdeps/powerpc/powerpc64/multiarch/bcopy-ppc64.c +++ b/sysdeps/powerpc/powerpc64/multiarch/bcopy-ppc64.c @@ -17,6 +17,7 @@ . */ #include +#include extern __typeof (bcopy) __bcopy_ppc attribute_hidden; extern __typeof (memmove) __memmove_ppc attribute_hidden; diff --git a/sysdeps/powerpc/powerpc64/multiarch/bcopy.c b/sysdeps/powerpc/powerpc64/multiarch/bcopy.c index 8c6f2fbf8e..b402117cb3 100644 --- a/sysdeps/powerpc/powerpc64/multiarch/bcopy.c +++ b/sysdeps/powerpc/powerpc64/multiarch/bcopy.c @@ -17,6 +17,7 @@ . */ #include +#include #include "init-arch.h" extern __typeof (bcopy) __bcopy_ppc attribute_hidden; diff --git a/sysdeps/powerpc/powerpc64/multiarch/ifunc-impl-list.c b/sysdeps/powerpc/powerpc64/multiarch/ifunc-impl-list.c index c0a927d73e..c6a227f0e1 100644 --- a/sysdeps/powerpc/powerpc64/multiarch/ifunc-impl-list.c +++ b/sysdeps/powerpc/powerpc64/multiarch/ifunc-impl-list.c @@ -18,6 +18,7 @@ #include #include +#include #include #include #include diff --git a/sysdeps/s390/bzero.c b/sysdeps/s390/bzero.c index 5477ed4aad..18f7eb8932 100644 --- a/sysdeps/s390/bzero.c +++ b/sysdeps/s390/bzero.c @@ -19,6 +19,7 @@ #include #if HAVE_MEMSET_IFUNC # include +# include # include # if HAVE_MEMSET_Z900_G5 diff --git a/sysdeps/s390/multiarch/ifunc-impl-list.c b/sysdeps/s390/multiarch/ifunc-impl-list.c index 1948436417..bede348ee5 100644 --- a/sysdeps/s390/multiarch/ifunc-impl-list.c +++ b/sysdeps/s390/multiarch/ifunc-impl-list.c @@ -18,6 +18,7 @@ #include #include +#include #include #include #include diff --git a/sysdeps/sparc/sparc64/multiarch/bzero.c b/sysdeps/sparc/sparc64/multiarch/bzero.c index 2d7c06b325..3801d8d0ef 100644 --- a/sysdeps/sparc/sparc64/multiarch/bzero.c +++ b/sysdeps/sparc/sparc64/multiarch/bzero.c @@ -20,6 +20,7 @@ #if IS_IN (libc) # define bzero __redirect_bzero # include +# include # undef bzero # include diff --git a/sysdeps/sparc/sparc64/multiarch/ifunc-impl-list.c b/sysdeps/sparc/sparc64/multiarch/ifunc-impl-list.c index 00ed640fbf..8326a7b421 100644 --- a/sysdeps/sparc/sparc64/multiarch/ifunc-impl-list.c +++ b/sysdeps/sparc/sparc64/multiarch/ifunc-impl-list.c @@ -18,6 +18,7 @@ #include #include +#include #include #include #include