aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJoseph Myers <joseph@codesourcery.com>2015-10-20 21:27:22 +0000
committerJoseph Myers <joseph@codesourcery.com>2015-10-20 21:27:22 +0000
commit80d9be8122d1fe181d2476b5dad9f4ce37204958 (patch)
tree4aacbba844669804ac4a3802a4001d5257350d11
parent9dd346ff431fc761f1b748bd4da8bb59f7652094 (diff)
downloadglibc-80d9be8122d1fe181d2476b5dad9f4ce37204958.tar.xz
glibc-80d9be8122d1fe181d2476b5dad9f4ce37204958.zip
Convert miscellaneous function definitions to prototype style.
This patch converts various miscellaneous functions definitions in glibc, found with grep and not covered by my previous scripted conversions, from old-style K&R to prototype-style. These changes were made manually. This is not necessarily exhaustive as formatting variants may have prevented my grep from finding some such definitions. Regarding the changes to files from GMP, they may originally have been omitted when removing __STDC__ conditionals because of the files coming from another package, but (a) GMP no longer has __STDC__ conditionals there anyway and (b) we don't try to keep these files verbatim in sync with GMP (and there are licensing differences), so making the change to them in glibc seems reasonable. Tested for x86_64 and x86 (testsuite - this patch affects files containing assertions). * debug/fortify_fail.c (__fortify_fail): Convert to prototype-style function definition. Use internal_function. * libio/genops.c (save_for_backup): Convert to prototype-style function definition. * libio/wgenops.c (save_for_wbackup): Likewise. * login/grantpt.c (grantpt): Likewise. * login/ptsname.c (ptsname): Likewise. (__ptsname_r): Likewise. * login/unlockpt.c (unlockpt): Likewise. * mach/msgserver.c (__mach_msg_server): Likewise. * misc/efgcvt.c (__APPEND (FUNC_PREFIX, fcvt)): Likewise. (__APPEND (FUNC_PREFIX, ecvt)): Likewise. (__APPEND (FUNC_PREFIX, gcvt)): Likewise. * misc/efgcvt_r.c (__APPEND (FUNC_PREFIX, fcvt_r)): Likewise. (__APPEND (FUNC_PREFIX, ecvt_r)): Likewise. * nptl/cleanup_compat.c (_pthread_cleanup_push): Likewise. * nptl/cleanup_defer_compat.c (_pthread_cleanup_push_defer): Likewise. * nptl/libc_pthread_init.c (__libc_pthread_init): Likewise. Use internal_function. * nptl/pthread_atfork.c (__pthread_atfork): Convert to prototype-style function definition. * nptl/pthread_create.c (__pthread_create_2_1): Likewise. [SHLIB_COMPAT(libpthread, GLIBC_2_0, GLIBC_2_1)] (__pthread_create_2_0): Likewise. * nptl/pthread_key_create.c (__pthread_key_create): Likewise. * nptl/register-atfork.c (__register_atfork): Likewise. * posix/glob.c (glob): Likewise. * posix/regcomp.c (re_comp): Likewise. * posix/regexec.c (re_exec): Likewise. * stdlib/add_n.c [__STDC__]: Make code unconditional. [!__STDC__]: Remove conditional code. * stdlib/cmp.c [__STDC__]: Make code unconditional. [!__STDC__]: Remove conditional code. * stdlib/divmod_1.c [__STDC__]: Make code unconditional. [!__STDC__]: Remove conditional code. * stdlib/divrem.c [__STDC__]: Make code unconditional. [!__STDC__]: Remove conditional code. * stdlib/lshift.c [__STDC__]: Make code unconditional. [!__STDC__]: Remove conditional code. * stdlib/mod_1.c [__STDC__]: Make code unconditional. [!__STDC__]: Remove conditional code. * stdlib/mul.c [__STDC__]: Make code unconditional. [!__STDC__]: Remove conditional code. * stdlib/mul_n.c [__STDC__]: Make code unconditional. [!__STDC__]: Remove conditional code. * stdlib/rshift.c [__STDC__]: Make code unconditional. [!__STDC__]: Remove conditional code. * stdlib/strtod.c (INTERNAL (STRTOF)): Convert to prototype-style function definition. (STRTOF): Likewise. * stdlib/strtod_l.c (__STRTOF): Likewise. * stdlib/strtol.c (INTERNAL (strtol)): Likewise. * stdlib/strtol_l.c (INTERNAL (__strtol_l)): Likewise. (__strtol_l): Likewise. * stdlib/sub_n.c [__STDC__]: Make code unconditional. [!__STDC__]: Remove conditional code. * string/memrchr.c (MEMRCHR): Convert to prototype-style function definition. * string/strcasecmp.c (LOCALE_PARAM_DECL): Remove macro. [USE_IN_EXTENDED_LOCALE_MODEL] (LOCALE_PARAM): Include argument type. (__strcasecmp): Convert to prototype-style function definition. * string/strncase.c (LOCALE_PARAM_DECL): Remove macro. [USE_IN_EXTENDED_LOCALE_MODEL] (LOCALE_PARAM): Include argument type. (__strncasecmp): Convert to prototype-style function definition. * sunrpc/pm_getport.c (__libc_rpc_getport): Likewise. * sunrpc/xdr.c (xdr_union): Likewise. * sunrpc/xdr_array.c (xdr_array): Likewise. * sunrpc/xdr_ref.c (xdr_reference): Likewise. * sysdeps/m68k/m680x0/fpu/s_atan.c (__CONCATX(__,FUNC)): Likewise. * sysdeps/m68k/m680x0/fpu/s_isinf.c (__CONCATX(__,FUNC)): Likewise. * sysdeps/m68k/m680x0/fpu/s_scalbn.c (__CONCATX(__scalbn,suffix): Likewise. * sysdeps/m68k/m680x0/fpu/s_sincos.c (CONCATX(__,FUNC)): Likewise. * sysdeps/unix/sysv/linux/i386/scandir64.c (__old_scandir64): Likewise. * time/strftime_l.c (LOCALE_PARAM_DECL): Remove macro. (LOCALE_PARAM_PROTO): Likewise. [_LIBC && USE_IN_EXTENDED_LOCALE_MODEL] (LOCALE_PARAM): Include argument type. (ut_argument_spec): Remove macro. (ut_argument_spec_iso): Rename to ut_argument_spec. (memcpy_lowcase): Use LOCALE_PARAM in declaration. Convert to prototype-style function definition. (memcpy_uppcase): Likewise. (__strftime_internal): Likewise. (my_strftime): Likewise. * time/strptime_l.c (LOCALE_PARAM_PROTO): Remove macro. (LOCALE_PARAM_DECL): Likewise. [_LIBC] (LOCALE_PARAM): Include argument type. (__strptime_internal): Convert to prototype-style function definition. (strptime): Likewise. * wcsmbs/wcscasecmp.c (LOCALE_PARAM_DECL): Remove macro. [USE_IN_EXTENDED_LOCALE_MODEL] (LOCALE_PARAM): Include argument type. (__wcscasecmp): Convert to prototype-style function definition. * wcsmbs/wcsncase.c (LOCALE_PARAM_DECL): Remove macro. [USE_IN_EXTENDED_LOCALE_MODEL] (LOCALE_PARAM): Include argument type. (__wcsncasecmp): Convert to prototype-style function definition.
-rw-r--r--ChangeLog105
-rw-r--r--debug/fortify_fail.c5
-rw-r--r--libio/genops.c4
-rw-r--r--libio/wgenops.c4
-rw-r--r--login/grantpt.c3
-rw-r--r--login/ptsname.c10
-rw-r--r--login/unlockpt.c3
-rw-r--r--mach/msgserver.c8
-rw-r--r--misc/efgcvt.c15
-rw-r--r--misc/efgcvt_r.c14
-rw-r--r--nptl/cleanup_compat.c6
-rw-r--r--nptl/cleanup_defer_compat.c6
-rw-r--r--nptl/libc_pthread_init.c7
-rw-r--r--nptl/pthread_atfork.c6
-rw-r--r--nptl/pthread_create.c14
-rw-r--r--nptl/pthread_key_create.c4
-rw-r--r--nptl/register-atfork.c7
-rw-r--r--posix/glob.c7
-rw-r--r--posix/regcomp.c3
-rw-r--r--posix/regexec.c3
-rw-r--r--stdlib/add_n.c8
-rw-r--r--stdlib/cmp.c7
-rw-r--r--stdlib/divmod_1.c8
-rw-r--r--stdlib/divrem.c10
-rw-r--r--stdlib/lshift.c8
-rw-r--r--stdlib/mod_1.c7
-rw-r--r--stdlib/mul.c9
-rw-r--r--stdlib/mul_n.c40
-rw-r--r--stdlib/rshift.c8
-rw-r--r--stdlib/strtod.c9
-rw-r--r--stdlib/strtod_l.c5
-rw-r--r--stdlib/strtol.c7
-rw-r--r--stdlib/strtol_l.c15
-rw-r--r--stdlib/sub_n.c8
-rw-r--r--string/memrchr.c5
-rw-r--r--string/strcasecmp.c9
-rw-r--r--string/strncase.c10
-rw-r--r--sunrpc/pm_getport.c11
-rw-r--r--sunrpc/xdr.c15
-rw-r--r--sunrpc/xdr_array.c18
-rw-r--r--sunrpc/xdr_ref.c12
-rw-r--r--sysdeps/m68k/m680x0/fpu/s_atan.c3
-rw-r--r--sysdeps/m68k/m680x0/fpu/s_isinf.c3
-rw-r--r--sysdeps/m68k/m680x0/fpu/s_scalbn.c4
-rw-r--r--sysdeps/m68k/m680x0/fpu/s_sincos.c3
-rw-r--r--sysdeps/unix/sysv/linux/i386/scandir64.c10
-rw-r--r--time/strftime_l.c51
-rw-r--r--time/strptime_l.c20
-rw-r--r--wcsmbs/wcscasecmp.c9
-rw-r--r--wcsmbs/wcsncase.c10
50 files changed, 221 insertions, 355 deletions
diff --git a/ChangeLog b/ChangeLog
index 9e7c455c56..d5ecc5d9c7 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,110 @@
2015-10-20 Joseph Myers <joseph@codesourcery.com>
+ * debug/fortify_fail.c (__fortify_fail): Convert to
+ prototype-style function definition. Use internal_function.
+ * libio/genops.c (save_for_backup): Convert to prototype-style
+ function definition.
+ * libio/wgenops.c (save_for_wbackup): Likewise.
+ * login/grantpt.c (grantpt): Likewise.
+ * login/ptsname.c (ptsname): Likewise.
+ (__ptsname_r): Likewise.
+ * login/unlockpt.c (unlockpt): Likewise.
+ * mach/msgserver.c (__mach_msg_server): Likewise.
+ * misc/efgcvt.c (__APPEND (FUNC_PREFIX, fcvt)): Likewise.
+ (__APPEND (FUNC_PREFIX, ecvt)): Likewise.
+ (__APPEND (FUNC_PREFIX, gcvt)): Likewise.
+ * misc/efgcvt_r.c (__APPEND (FUNC_PREFIX, fcvt_r)): Likewise.
+ (__APPEND (FUNC_PREFIX, ecvt_r)): Likewise.
+ * nptl/cleanup_compat.c (_pthread_cleanup_push): Likewise.
+ * nptl/cleanup_defer_compat.c (_pthread_cleanup_push_defer):
+ Likewise.
+ * nptl/libc_pthread_init.c (__libc_pthread_init): Likewise. Use
+ internal_function.
+ * nptl/pthread_atfork.c (__pthread_atfork): Convert to
+ prototype-style function definition.
+ * nptl/pthread_create.c (__pthread_create_2_1): Likewise.
+ [SHLIB_COMPAT(libpthread, GLIBC_2_0, GLIBC_2_1)]
+ (__pthread_create_2_0): Likewise.
+ * nptl/pthread_key_create.c (__pthread_key_create): Likewise.
+ * nptl/register-atfork.c (__register_atfork): Likewise.
+ * posix/glob.c (glob): Likewise.
+ * posix/regcomp.c (re_comp): Likewise.
+ * posix/regexec.c (re_exec): Likewise.
+ * stdlib/add_n.c [__STDC__]: Make code unconditional.
+ [!__STDC__]: Remove conditional code.
+ * stdlib/cmp.c [__STDC__]: Make code unconditional.
+ [!__STDC__]: Remove conditional code.
+ * stdlib/divmod_1.c [__STDC__]: Make code unconditional.
+ [!__STDC__]: Remove conditional code.
+ * stdlib/divrem.c [__STDC__]: Make code unconditional.
+ [!__STDC__]: Remove conditional code.
+ * stdlib/lshift.c [__STDC__]: Make code unconditional.
+ [!__STDC__]: Remove conditional code.
+ * stdlib/mod_1.c [__STDC__]: Make code unconditional.
+ [!__STDC__]: Remove conditional code.
+ * stdlib/mul.c [__STDC__]: Make code unconditional.
+ [!__STDC__]: Remove conditional code.
+ * stdlib/mul_n.c [__STDC__]: Make code unconditional.
+ [!__STDC__]: Remove conditional code.
+ * stdlib/rshift.c [__STDC__]: Make code unconditional.
+ [!__STDC__]: Remove conditional code.
+ * stdlib/strtod.c (INTERNAL (STRTOF)): Convert to prototype-style
+ function definition.
+ (STRTOF): Likewise.
+ * stdlib/strtod_l.c (__STRTOF): Likewise.
+ * stdlib/strtol.c (INTERNAL (strtol)): Likewise.
+ * stdlib/strtol_l.c (INTERNAL (__strtol_l)): Likewise.
+ (__strtol_l): Likewise.
+ * stdlib/sub_n.c [__STDC__]: Make code unconditional.
+ [!__STDC__]: Remove conditional code.
+ * string/memrchr.c (MEMRCHR): Convert to prototype-style function
+ definition.
+ * string/strcasecmp.c (LOCALE_PARAM_DECL): Remove macro.
+ [USE_IN_EXTENDED_LOCALE_MODEL] (LOCALE_PARAM): Include argument
+ type.
+ (__strcasecmp): Convert to prototype-style function definition.
+ * string/strncase.c (LOCALE_PARAM_DECL): Remove macro.
+ [USE_IN_EXTENDED_LOCALE_MODEL] (LOCALE_PARAM): Include argument
+ type.
+ (__strncasecmp): Convert to prototype-style function definition.
+ * sunrpc/pm_getport.c (__libc_rpc_getport): Likewise.
+ * sunrpc/xdr.c (xdr_union): Likewise.
+ * sunrpc/xdr_array.c (xdr_array): Likewise.
+ * sunrpc/xdr_ref.c (xdr_reference): Likewise.
+ * sysdeps/m68k/m680x0/fpu/s_atan.c (__CONCATX(__,FUNC)): Likewise.
+ * sysdeps/m68k/m680x0/fpu/s_isinf.c (__CONCATX(__,FUNC)):
+ Likewise.
+ * sysdeps/m68k/m680x0/fpu/s_scalbn.c (__CONCATX(__scalbn,suffix):
+ Likewise.
+ * sysdeps/m68k/m680x0/fpu/s_sincos.c (CONCATX(__,FUNC)): Likewise.
+ * sysdeps/unix/sysv/linux/i386/scandir64.c (__old_scandir64):
+ Likewise.
+ * time/strftime_l.c (LOCALE_PARAM_DECL): Remove macro.
+ (LOCALE_PARAM_PROTO): Likewise.
+ [_LIBC && USE_IN_EXTENDED_LOCALE_MODEL] (LOCALE_PARAM): Include
+ argument type.
+ (ut_argument_spec): Remove macro.
+ (ut_argument_spec_iso): Rename to ut_argument_spec.
+ (memcpy_lowcase): Use LOCALE_PARAM in declaration. Convert to
+ prototype-style function definition.
+ (memcpy_uppcase): Likewise.
+ (__strftime_internal): Likewise.
+ (my_strftime): Likewise.
+ * time/strptime_l.c (LOCALE_PARAM_PROTO): Remove macro.
+ (LOCALE_PARAM_DECL): Likewise.
+ [_LIBC] (LOCALE_PARAM): Include argument type.
+ (__strptime_internal): Convert to prototype-style function
+ definition.
+ (strptime): Likewise.
+ * wcsmbs/wcscasecmp.c (LOCALE_PARAM_DECL): Remove macro.
+ [USE_IN_EXTENDED_LOCALE_MODEL] (LOCALE_PARAM): Include argument
+ type.
+ (__wcscasecmp): Convert to prototype-style function definition.
+ * wcsmbs/wcsncase.c (LOCALE_PARAM_DECL): Remove macro.
+ [USE_IN_EXTENDED_LOCALE_MODEL] (LOCALE_PARAM): Include argument
+ type.
+ (__wcsncasecmp): Convert to prototype-style function definition.
+
* crypt/md5-crypt.c (__md5_crypt_r): Convert to prototype-style
function definition.
* crypt/sha256-crypt.c (__sha256_crypt_r): Likewise.
diff --git a/debug/fortify_fail.c b/debug/fortify_fail.c
index f992695d14..d4f98da008 100644
--- a/debug/fortify_fail.c
+++ b/debug/fortify_fail.c
@@ -22,9 +22,8 @@
extern char **__libc_argv attribute_hidden;
void
-__attribute__ ((noreturn))
-__fortify_fail (msg)
- const char *msg;
+__attribute__ ((noreturn)) internal_function
+__fortify_fail (const char *msg)
{
/* The loop is added only to keep gcc happy. */
while (1)
diff --git a/libio/genops.c b/libio/genops.c
index 377bda311f..32fec76648 100644
--- a/libio/genops.c
+++ b/libio/genops.c
@@ -243,9 +243,7 @@ static int
#ifdef _LIBC
internal_function
#endif
-save_for_backup (fp, end_p)
- _IO_FILE *fp;
- char *end_p;
+save_for_backup (_IO_FILE *fp, char *end_p)
{
/* Append [_IO_read_base..end_p] to backup area. */
_IO_ssize_t least_mark = _IO_least_marker (fp, end_p);
diff --git a/libio/wgenops.c b/libio/wgenops.c
index e038f2c8bf..5c7cdc523b 100644
--- a/libio/wgenops.c
+++ b/libio/wgenops.c
@@ -469,9 +469,7 @@ static int
#ifdef _LIBC
internal_function
#endif
-save_for_wbackup (fp, end_p)
- _IO_FILE *fp;
- wchar_t *end_p;
+save_for_wbackup (_IO_FILE *fp, wchar_t *end_p)
{
/* Append [_IO_read_base..end_p] to backup area. */
_IO_ssize_t least_mark = _IO_least_wmarker (fp, end_p);
diff --git a/login/grantpt.c b/login/grantpt.c
index eb48342bbd..d90c757bd5 100644
--- a/login/grantpt.c
+++ b/login/grantpt.c
@@ -23,8 +23,7 @@
with the slave to the calling process, and set its group and
mode appropriately. Note that this is an unprivileged operation. */
int
-grantpt (fd)
- int fd __attribute__ ((unused));
+grantpt (int fd __attribute__ ((unused)))
{
__set_errno (ENOSYS);
return -1;
diff --git a/login/ptsname.c b/login/ptsname.c
index 9536b05698..4fe138d5b0 100644
--- a/login/ptsname.c
+++ b/login/ptsname.c
@@ -24,18 +24,16 @@
of the associated slave. */
char *
-ptsname (fd)
- int fd __attribute__ ((unused));
+ptsname (int fd __attribute__ ((unused)))
{
__set_errno (ENOSYS);
return NULL;
}
int
-__ptsname_r (fd, buf, len)
- int fd __attribute__ ((unused));
- char *buf __attribute__ ((unused));
- size_t len __attribute__ ((unused));
+__ptsname_r (int fd __attribute__ ((unused)),
+ char *buf __attribute__ ((unused)),
+ size_t len __attribute__ ((unused)))
{
__set_errno (ENOSYS);
return ENOSYS;
diff --git a/login/unlockpt.c b/login/unlockpt.c
index 1f3111e72e..595c3f4efd 100644
--- a/login/unlockpt.c
+++ b/login/unlockpt.c
@@ -23,8 +23,7 @@
the slave can be opened. This is to avoid a race between opening the
master and calling grantpt() to take possession of the slave. */
int
-unlockpt (fd)
- int fd __attribute__ ((unused));
+unlockpt (int fd __attribute__ ((unused)))
{
__set_errno (ENOSYS);
return -1;
diff --git a/mach/msgserver.c b/mach/msgserver.c
index 2f4a08ee3a..d90b6c5711 100644
--- a/mach/msgserver.c
+++ b/mach/msgserver.c
@@ -187,10 +187,10 @@ __mach_msg_server_timeout (boolean_t (*demux) (mach_msg_header_t *request,
weak_alias (__mach_msg_server_timeout, mach_msg_server_timeout)
mach_msg_return_t
-__mach_msg_server (demux, max_size, rcv_name)
- boolean_t (*demux) (mach_msg_header_t *in, mach_msg_header_t *out);
- mach_msg_size_t max_size;
-