aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJoseph Myers <joseph@codesourcery.com>2019-02-25 13:19:19 +0000
committerJoseph Myers <joseph@codesourcery.com>2019-02-25 13:19:19 +0000
commita04549c19407a29a271779598a9518f9baf959e0 (patch)
treed594a725127d2f60112308b3debd2c7b7c2db085
parent34a5a1460e9c05d6035bfbde327ab6d45f78958b (diff)
downloadglibc-a04549c19407a29a271779598a9518f9baf959e0.tar.xz
glibc-a04549c19407a29a271779598a9518f9baf959e0.zip
Break more lines before not after operators.
This patch makes further coding style fixes where code was breaking lines after an operator, contrary to the GNU Coding Standards. As with the previous patch, it is limited to files following a reasonable approximation to GNU style already, and is not exhaustive; more such issues remain to be fixed. Tested for x86_64, and with build-many-glibcs.py. * dirent/dirent.h [!_DIRENT_HAVE_D_NAMLEN && _DIRENT_HAVE_D_RECLEN] (_D_ALLOC_NAMLEN): Break lines before rather than after operators. * elf/cache.c (print_cache): Likewise. * gshadow/fgetsgent_r.c (__fgetsgent_r): Likewise. * htl/pt-getattr.c (__pthread_getattr_np): Likewise. * hurd/hurdinit.c (_hurd_setproc): Likewise. * hurd/hurdkill.c (_hurd_sig_post): Likewise. * hurd/hurdlookup.c (__file_name_lookup_under): Likewise. * hurd/hurdsig.c (_hurd_internal_post_signal): Likewise. (reauth_proc): Likewise. * hurd/lookup-at.c (__file_name_lookup_at): Likewise. (__file_name_split_at): Likewise. (__directory_name_split_at): Likewise. * hurd/lookup-retry.c (__hurd_file_name_lookup_retry): Likewise. * hurd/port2fd.c (_hurd_port2fd): Likewise. * iconv/gconv_dl.c (do_print): Likewise. * inet/netinet/in.h (struct sockaddr_in): Likewise. * libio/wstrops.c (_IO_wstr_seekoff): Likewise. * locale/setlocale.c (new_composite_name): Likewise. * malloc/memusagestat.c (main): Likewise. * misc/fstab.c (fstab_convert): Likewise. * nptl/pthread_mutex_unlock.c (__pthread_mutex_unlock_usercnt): Likewise. * nss/nss_compat/compat-grp.c (getgrent_next_nss): Likewise. (getgrent_next_file): Likewise. (internal_getgrnam_r): Likewise. (internal_getgrgid_r): Likewise. * nss/nss_compat/compat-initgroups.c (getgrent_next_nss): Likewise. (internal_getgrent_r): Likewise. * nss/nss_compat/compat-pwd.c (getpwent_next_nss_netgr): Likewise. (getpwent_next_nss): Likewise. (getpwent_next_file): Likewise. (internal_getpwnam_r): Likewise. (internal_getpwuid_r): Likewise. * nss/nss_compat/compat-spwd.c (getspent_next_nss_netgr): Likewise. (getspent_next_nss): Likewise. (internal_getspnam_r): Likewise. * pwd/fgetpwent_r.c (__fgetpwent_r): Likewise. * shadow/fgetspent_r.c (__fgetspent_r): Likewise. * string/strchr.c (STRCHR): Likewise. * string/strchrnul.c (STRCHRNUL): Likewise. * sysdeps/aarch64/fpu/fpu_control.h (_FPU_FPCR_IEEE): Likewise. * sysdeps/aarch64/sfp-machine.h (_FP_CHOOSENAN): Likewise. * sysdeps/csky/dl-machine.h (elf_machine_rela): Likewise. * sysdeps/generic/memcopy.h (PAGE_COPY_FWD_MAYBE): Likewise. * sysdeps/generic/symbol-hacks.h (__stack_chk_fail_local): Likewise. * sysdeps/gnu/netinet/ip_icmp.h (ICMP_INFOTYPE): Likewise. * sysdeps/gnu/updwtmp.c (TRANSFORM_UTMP_FILE_NAME): Likewise. * sysdeps/gnu/utmp_file.c (TRANSFORM_UTMP_FILE_NAME): Likewise. * sysdeps/hppa/jmpbuf-unwind.h (_JMPBUF_UNWINDS): Likewise. * sysdeps/mach/hurd/bits/stat.h (S_ISPARE): Likewise. * sysdeps/mach/hurd/dl-sysdep.c (_dl_sysdep_start): Likewise. (open_file): Likewise. * sysdeps/mach/hurd/htl/pt-mutexattr-setprotocol.c (pthread_mutexattr_setprotocol): Likewise. * sysdeps/mach/hurd/ioctl.c (__ioctl): Likewise. * sysdeps/mach/hurd/mmap.c (__mmap): Likewise. * sysdeps/mach/hurd/ptrace.c (ptrace): Likewise. * sysdeps/mach/hurd/spawni.c (__spawni): Likewise. * sysdeps/microblaze/dl-machine.h (elf_machine_type_class): Likewise. (elf_machine_rela): Likewise. * sysdeps/mips/mips32/sfp-machine.h (_FP_CHOOSENAN): Likewise. * sysdeps/mips/mips64/sfp-machine.h (_FP_CHOOSENAN): Likewise. * sysdeps/mips/sys/asm.h (multiple #if conditionals): Likewise. * sysdeps/posix/rename.c (rename): Likewise. * sysdeps/powerpc/novmx-sigjmp.c (__novmx__sigjmp_save): Likewise. * sysdeps/powerpc/sigjmp.c (__vmx__sigjmp_save): Likewise. * sysdeps/s390/fpu/fenv_libc.h (FPC_VALID_MASK): Likewise. * sysdeps/s390/utf8-utf16-z9.c (gconv_end): Likewise. * sysdeps/unix/grantpt.c (grantpt): Likewise. * sysdeps/unix/sysv/linux/a.out.h (N_TXTOFF): Likewise. * sysdeps/unix/sysv/linux/updwtmp.c (TRANSFORM_UTMP_FILE_NAME): Likewise. * sysdeps/unix/sysv/linux/utmp_file.c (TRANSFORM_UTMP_FILE_NAME): Likewise. * sysdeps/x86/cpu-features.c (get_common_indices): Likewise. * time/tzfile.c (__tzfile_compute): Likewise.
-rw-r--r--ChangeLog85
-rw-r--r--dirent/dirent.h4
-rw-r--r--elf/cache.c4
-rw-r--r--gshadow/fgetsgent_r.c6
-rw-r--r--htl/pt-getattr.c6
-rw-r--r--hurd/hurdinit.c8
-rw-r--r--hurd/hurdkill.c4
-rw-r--r--hurd/hurdlookup.c4
-rw-r--r--hurd/hurdsig.c24
-rw-r--r--hurd/lookup-at.c12
-rw-r--r--hurd/lookup-retry.c8
-rw-r--r--hurd/port2fd.c5
-rw-r--r--iconv/gconv_dl.c6
-rw-r--r--inet/netinet/in.h8
-rw-r--r--libio/wstrops.c4
-rw-r--r--locale/setlocale.c12
-rw-r--r--malloc/memusagestat.c4
-rw-r--r--misc/fstab.c12
-rw-r--r--nptl/pthread_mutex_unlock.c5
-rw-r--r--nss/nss_compat/compat-grp.c22
-rw-r--r--nss/nss_compat/compat-initgroups.c10
-rw-r--r--nss/nss_compat/compat-pwd.c26
-rw-r--r--nss/nss_compat/compat-spwd.c14
-rw-r--r--pwd/fgetpwent_r.c4
-rw-r--r--shadow/fgetspent_r.c4
-rw-r--r--string/strchr.c6
-rw-r--r--string/strchrnul.c6
-rw-r--r--sysdeps/aarch64/fpu/fpu_control.h6
-rw-r--r--sysdeps/aarch64/sfp-machine.h4
-rw-r--r--sysdeps/csky/dl-machine.h4
-rw-r--r--sysdeps/generic/memcopy.h4
-rw-r--r--sysdeps/generic/symbol-hacks.h4
-rw-r--r--sysdeps/gnu/netinet/ip_icmp.h10
-rw-r--r--sysdeps/gnu/updwtmp.c14
-rw-r--r--sysdeps/gnu/utmp_file.c14
-rw-r--r--sysdeps/hppa/jmpbuf-unwind.h4
-rw-r--r--sysdeps/mach/hurd/bits/stat.h4
-rw-r--r--sysdeps/mach/hurd/dl-sysdep.c8
-rw-r--r--sysdeps/mach/hurd/htl/pt-mutexattr-setprotocol.c7
-rw-r--r--sysdeps/mach/hurd/ioctl.c6
-rw-r--r--sysdeps/mach/hurd/mmap.c4
-rw-r--r--sysdeps/mach/hurd/ptrace.c4
-rw-r--r--sysdeps/mach/hurd/spawni.c4
-rw-r--r--sysdeps/microblaze/dl-machine.h14
-rw-r--r--sysdeps/mips/mips32/sfp-machine.h4
-rw-r--r--sysdeps/mips/mips64/sfp-machine.h4
-rw-r--r--sysdeps/mips/sys/asm.h16
-rw-r--r--sysdeps/posix/rename.c4
-rw-r--r--sysdeps/powerpc/novmx-sigjmp.c6
-rw-r--r--sysdeps/powerpc/sigjmp.c6
-rw-r--r--sysdeps/s390/fpu/fenv_libc.h4
-rw-r--r--sysdeps/s390/utf8-utf16-z9.c4
-rw-r--r--sysdeps/unix/grantpt.c4
-rw-r--r--sysdeps/unix/sysv/linux/a.out.h4
-rw-r--r--sysdeps/unix/sysv/linux/updwtmp.c24
-rw-r--r--sysdeps/unix/sysv/linux/utmp_file.c24
-rw-r--r--sysdeps/x86/cpu-features.c8
-rw-r--r--time/tzfile.c6
58 files changed, 323 insertions, 223 deletions
diff --git a/ChangeLog b/ChangeLog
index aaf966c1e7..08c2dc6d0d 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,88 @@
+2019-02-25 Joseph Myers <joseph@codesourcery.com>
+
+ * dirent/dirent.h [!_DIRENT_HAVE_D_NAMLEN
+ && _DIRENT_HAVE_D_RECLEN] (_D_ALLOC_NAMLEN): Break lines before
+ rather than after operators.
+ * elf/cache.c (print_cache): Likewise.
+ * gshadow/fgetsgent_r.c (__fgetsgent_r): Likewise.
+ * htl/pt-getattr.c (__pthread_getattr_np): Likewise.
+ * hurd/hurdinit.c (_hurd_setproc): Likewise.
+ * hurd/hurdkill.c (_hurd_sig_post): Likewise.
+ * hurd/hurdlookup.c (__file_name_lookup_under): Likewise.
+ * hurd/hurdsig.c (_hurd_internal_post_signal): Likewise.
+ (reauth_proc): Likewise.
+ * hurd/lookup-at.c (__file_name_lookup_at): Likewise.
+ (__file_name_split_at): Likewise.
+ (__directory_name_split_at): Likewise.
+ * hurd/lookup-retry.c (__hurd_file_name_lookup_retry): Likewise.
+ * hurd/port2fd.c (_hurd_port2fd): Likewise.
+ * iconv/gconv_dl.c (do_print): Likewise.
+ * inet/netinet/in.h (struct sockaddr_in): Likewise.
+ * libio/wstrops.c (_IO_wstr_seekoff): Likewise.
+ * locale/setlocale.c (new_composite_name): Likewise.
+ * malloc/memusagestat.c (main): Likewise.
+ * misc/fstab.c (fstab_convert): Likewise.
+ * nptl/pthread_mutex_unlock.c (__pthread_mutex_unlock_usercnt):
+ Likewise.
+ * nss/nss_compat/compat-grp.c (getgrent_next_nss): Likewise.
+ (getgrent_next_file): Likewise.
+ (internal_getgrnam_r): Likewise.
+ (internal_getgrgid_r): Likewise.
+ * nss/nss_compat/compat-initgroups.c (getgrent_next_nss):
+ Likewise.
+ (internal_getgrent_r): Likewise.
+ * nss/nss_compat/compat-pwd.c (getpwent_next_nss_netgr): Likewise.
+ (getpwent_next_nss): Likewise.
+ (getpwent_next_file): Likewise.
+ (internal_getpwnam_r): Likewise.
+ (internal_getpwuid_r): Likewise.
+ * nss/nss_compat/compat-spwd.c (getspent_next_nss_netgr):
+ Likewise.
+ (getspent_next_nss): Likewise.
+ (internal_getspnam_r): Likewise.
+ * pwd/fgetpwent_r.c (__fgetpwent_r): Likewise.
+ * shadow/fgetspent_r.c (__fgetspent_r): Likewise.
+ * string/strchr.c (STRCHR): Likewise.
+ * string/strchrnul.c (STRCHRNUL): Likewise.
+ * sysdeps/aarch64/fpu/fpu_control.h (_FPU_FPCR_IEEE): Likewise.
+ * sysdeps/aarch64/sfp-machine.h (_FP_CHOOSENAN): Likewise.
+ * sysdeps/csky/dl-machine.h (elf_machine_rela): Likewise.
+ * sysdeps/generic/memcopy.h (PAGE_COPY_FWD_MAYBE): Likewise.
+ * sysdeps/generic/symbol-hacks.h (__stack_chk_fail_local):
+ Likewise.
+ * sysdeps/gnu/netinet/ip_icmp.h (ICMP_INFOTYPE): Likewise.
+ * sysdeps/gnu/updwtmp.c (TRANSFORM_UTMP_FILE_NAME): Likewise.
+ * sysdeps/gnu/utmp_file.c (TRANSFORM_UTMP_FILE_NAME): Likewise.
+ * sysdeps/hppa/jmpbuf-unwind.h (_JMPBUF_UNWINDS): Likewise.
+ * sysdeps/mach/hurd/bits/stat.h (S_ISPARE): Likewise.
+ * sysdeps/mach/hurd/dl-sysdep.c (_dl_sysdep_start): Likewise.
+ (open_file): Likewise.
+ * sysdeps/mach/hurd/htl/pt-mutexattr-setprotocol.c
+ (pthread_mutexattr_setprotocol): Likewise.
+ * sysdeps/mach/hurd/ioctl.c (__ioctl): Likewise.
+ * sysdeps/mach/hurd/mmap.c (__mmap): Likewise.
+ * sysdeps/mach/hurd/ptrace.c (ptrace): Likewise.
+ * sysdeps/mach/hurd/spawni.c (__spawni): Likewise.
+ * sysdeps/microblaze/dl-machine.h (elf_machine_type_class):
+ Likewise.
+ (elf_machine_rela): Likewise.
+ * sysdeps/mips/mips32/sfp-machine.h (_FP_CHOOSENAN): Likewise.
+ * sysdeps/mips/mips64/sfp-machine.h (_FP_CHOOSENAN): Likewise.
+ * sysdeps/mips/sys/asm.h (multiple #if conditionals): Likewise.
+ * sysdeps/posix/rename.c (rename): Likewise.
+ * sysdeps/powerpc/novmx-sigjmp.c (__novmx__sigjmp_save): Likewise.
+ * sysdeps/powerpc/sigjmp.c (__vmx__sigjmp_save): Likewise.
+ * sysdeps/s390/fpu/fenv_libc.h (FPC_VALID_MASK): Likewise.
+ * sysdeps/s390/utf8-utf16-z9.c (gconv_end): Likewise.
+ * sysdeps/unix/grantpt.c (grantpt): Likewise.
+ * sysdeps/unix/sysv/linux/a.out.h (N_TXTOFF): Likewise.
+ * sysdeps/unix/sysv/linux/updwtmp.c (TRANSFORM_UTMP_FILE_NAME):
+ Likewise.
+ * sysdeps/unix/sysv/linux/utmp_file.c (TRANSFORM_UTMP_FILE_NAME):
+ Likewise.
+ * sysdeps/x86/cpu-features.c (get_common_indices): Likewise.
+ * time/tzfile.c (__tzfile_compute): Likewise.
+
2019-02-22 Joseph Myers <joseph@codesourcery.com>
* benchtests/bench-memmem.c (simple_memmem): Break lines before
diff --git a/dirent/dirent.h b/dirent/dirent.h
index 9494360ce8..fc4d4859fe 100644
--- a/dirent/dirent.h
+++ b/dirent/dirent.h
@@ -86,8 +86,8 @@ typedef __ino64_t ino64_t;
# ifdef _DIRENT_HAVE_D_RECLEN
# define _D_ALLOC_NAMLEN(d) (((char *) (d) + (d)->d_reclen) - &(d)->d_name[0])
# else
-# define _D_ALLOC_NAMLEN(d) (sizeof (d)->d_name > 1 ? sizeof (d)->d_name : \
- _D_EXACT_NAMLEN (d) + 1)
+# define _D_ALLOC_NAMLEN(d) (sizeof (d)->d_name > 1 ? sizeof (d)->d_name \
+ : _D_EXACT_NAMLEN (d) + 1)
# endif
#endif
diff --git a/elf/cache.c b/elf/cache.c
index b8934898b6..a64d67ecbe 100644
--- a/elf/cache.c
+++ b/elf/cache.c
@@ -211,8 +211,8 @@ print_cache (const char *cache_name)
cache_data = (const char *) &cache->libs[cache->nlibs];
/* Check for a new cache embedded in the old format. */
- if (cache_size >
- (offset + sizeof (struct cache_file_new)))
+ if (cache_size
+ > (offset + sizeof (struct cache_file_new)))
{
cache_new = (struct cache_file_new *) ((void *)cache + offset);
diff --git a/gshadow/fgetsgent_r.c b/gshadow/fgetsgent_r.c
index 7ecfb217e3..19a663f048 100644
--- a/gshadow/fgetsgent_r.c
+++ b/gshadow/fgetsgent_r.c
@@ -61,11 +61,11 @@ __fgetsgent_r (FILE *stream, struct sgrp *resbuf, char *buffer, size_t buflen,
/* Skip leading blanks. */
while (isspace (*p))
++p;
- } while (*p == '\0' || *p == '#' || /* Ignore empty and comment lines. */
+ } while (*p == '\0' || *p == '#' /* Ignore empty and comment lines. */
/* Parse the line. If it is invalid, loop to
get the next line of the file to parse. */
- ! parse_line (buffer, (void *) resbuf, (void *) buffer, buflen,
- &errno));
+ || ! parse_line (buffer, (void *) resbuf, (void *) buffer, buflen,
+ &errno));
_IO_funlockfile (stream);
diff --git a/htl/pt-getattr.c b/htl/pt-getattr.c
index 7fb4a9f6e8..33fd87a9fb 100644
--- a/htl/pt-getattr.c
+++ b/htl/pt-getattr.c
@@ -38,9 +38,9 @@ __pthread_getattr_np (pthread_t thread, pthread_attr_t *attr)
are not supported yet, so fill them with our default values. */
*attr = __pthread_default_attr;
- attr->__stackaddr = pthread->stackaddr +
- ((pthread->guardsize + __vm_page_size - 1)
- / __vm_page_size * __vm_page_size);
+ attr->__stackaddr = (pthread->stackaddr
+ + ((pthread->guardsize + __vm_page_size - 1)
+ / __vm_page_size * __vm_page_size));
attr->__stacksize = pthread->stacksize;
attr->__guardsize = pthread->guardsize;
attr->__detachstate = (pthread->state == PTHREAD_DETACHED
diff --git a/hurd/hurdinit.c b/hurd/hurdinit.c
index 773bb1c37f..f4a3607ae6 100644
--- a/hurd/hurdinit.c
+++ b/hurd/hurdinit.c
@@ -198,10 +198,10 @@ _hurd_setproc (process_t procserver)
/* Tell the proc server where our args and environment are. */
if (err = __proc_set_arg_locations (procserver,
- _hide_arguments ? 0 :
- (vm_address_t) __libc_argv,
- _hide_environment ? 0 :
- (vm_address_t) __environ))
+ _hide_arguments ? 0
+ : (vm_address_t) __libc_argv,
+ _hide_environment ? 0