This is the mail archive of the
glibc-cvs@sourceware.org
mailing list for the glibc project.
GNU C Library master sources branch master updated. glibc-2.29.9000-117-gc5f6546
- From: jsm28 at sourceware dot org
- To: glibc-cvs at sourceware dot org
- Date: 7 Mar 2019 20:20:53 -0000
- Subject: GNU C Library master sources branch master updated. glibc-2.29.9000-117-gc5f6546
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU C Library master sources".
The branch, master has been updated
via c5f65462a29fa4b7caa9026b36b2792d79b16d1d (commit)
from 27a2f2f34c9e77815c6367cf670e24cbac0db7c0 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
http://sourceware.org/git/gitweb.cgi?p=glibc.git;a=commitdiff;h=c5f65462a29fa4b7caa9026b36b2792d79b16d1d
commit c5f65462a29fa4b7caa9026b36b2792d79b16d1d
Author: Joseph Myers <joseph@codesourcery.com>
Date: Thu Mar 7 20:20:25 2019 +0000
Break lines before not after operators, batch 4.
This patch fixes further coding style issues where code should have
broken lines before operators in accordance with the GNU Coding
Standards but instead was breaking lines after them.
Tested for x86_64, and with build-many-glibcs.py.
* stdio-common/vfscanf-internal.c (ARG): Break lines before rather
than after operators.
* sysdeps/mach/hurd/setitimer.c (timer_thread): Likewise.
(setitimer_locked): Likewise.
* sysdeps/mach/hurd/sigaction.c (__sigaction): Likewise.
* sysdeps/mach/hurd/sigaltstack.c (__sigaltstack): Likewise.
* sysdeps/mach/pagecopy.h (PAGE_COPY_FWD): Likewise.
* sysdeps/mach/thread_state.h (machine_get_basic_state): Likewise.
* sysdeps/powerpc/powerpc64/tst-ucontext-ppc64-vscr.c
(PPC_CPU_SUPPORTED): Likewise.
* sysdeps/unix/sysv/linux/alpha/a.out.h (N_TXTOFF): Likewise.
* sysdeps/unix/sysv/linux/generic/wordsize-32/overflow.h
(stat_overflow): Likewise.
(statfs_overflow): Likewise.
* sysdeps/unix/sysv/linux/tst-personality.c (do_test): Likewise.
* sysdeps/unix/sysv/linux/tst-ttyname.c (eq_ttyname): Likewise.
(eq_ttyname_r): Likewise.
(run_chroot_tests): Likewise.
diff --git a/ChangeLog b/ChangeLog
index c657d79..a732f25 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,24 @@
+2019-03-07 Joseph Myers <joseph@codesourcery.com>
+
+ * stdio-common/vfscanf-internal.c (ARG): Break lines before rather
+ than after operators.
+ * sysdeps/mach/hurd/setitimer.c (timer_thread): Likewise.
+ (setitimer_locked): Likewise.
+ * sysdeps/mach/hurd/sigaction.c (__sigaction): Likewise.
+ * sysdeps/mach/hurd/sigaltstack.c (__sigaltstack): Likewise.
+ * sysdeps/mach/pagecopy.h (PAGE_COPY_FWD): Likewise.
+ * sysdeps/mach/thread_state.h (machine_get_basic_state): Likewise.
+ * sysdeps/powerpc/powerpc64/tst-ucontext-ppc64-vscr.c
+ (PPC_CPU_SUPPORTED): Likewise.
+ * sysdeps/unix/sysv/linux/alpha/a.out.h (N_TXTOFF): Likewise.
+ * sysdeps/unix/sysv/linux/generic/wordsize-32/overflow.h
+ (stat_overflow): Likewise.
+ (statfs_overflow): Likewise.
+ * sysdeps/unix/sysv/linux/tst-personality.c (do_test): Likewise.
+ * sysdeps/unix/sysv/linux/tst-ttyname.c (eq_ttyname): Likewise.
+ (eq_ttyname_r): Likewise.
+ (run_chroot_tests): Likewise.
+
2019-03-07 Florian Weimer <fweimer@redhat.com>
* scripts/check-wrapper-headers.py (check_headers): Adjust Fortran
diff --git a/stdio-common/vfscanf-internal.c b/stdio-common/vfscanf-internal.c
index 888b2b1..f43ceb1 100644
--- a/stdio-common/vfscanf-internal.c
+++ b/stdio-common/vfscanf-internal.c
@@ -385,32 +385,32 @@ __vfscanf_internal (FILE *s, const char *format, va_list argptr,
For a %N$... spec, this is the Nth argument from the beginning;
otherwise it is the next argument after the state now in ARG. */
#ifdef __va_copy
-# define ARG(type) (argpos == 0 ? va_arg (arg, type) : \
- ({ unsigned int pos = argpos; \
- va_list arg; \
- __va_copy (arg, argptr); \
- while (--pos > 0) \
- (void) va_arg (arg, void *); \
- va_arg (arg, type); \
- }))
+# define ARG(type) (argpos == 0 ? va_arg (arg, type) \
+ : ({ unsigned int pos = argpos; \
+ va_list arg; \
+ __va_copy (arg, argptr); \
+ while (--pos > 0) \
+ (void) va_arg (arg, void *); \
+ va_arg (arg, type); \
+ }))
#else
# if 0
/* XXX Possible optimization. */
-# define ARG(type) (argpos == 0 ? va_arg (arg, type) : \
- ({ va_list arg = (va_list) argptr; \
- arg = (va_list) ((char *) arg \
- + (argpos - 1) \
- * __va_rounded_size (void *)); \
- va_arg (arg, type); \
- }))
+# define ARG(type) (argpos == 0 ? va_arg (arg, type) \
+ : ({ va_list arg = (va_list) argptr; \
+ arg = (va_list) ((char *) arg \
+ + (argpos - 1) \
+ * __va_rounded_size (void *)); \
+ va_arg (arg, type); \
+ }))
# else
-# define ARG(type) (argpos == 0 ? va_arg (arg, type) : \
- ({ unsigned int pos = argpos; \
- va_list arg = (va_list) argptr; \
- while (--pos > 0) \
- (void) va_arg (arg, void *); \
- va_arg (arg, type); \
- }))
+# define ARG(type) (argpos == 0 ? va_arg (arg, type) \
+ : ({ unsigned int pos = argpos; \
+ va_list arg = (va_list) argptr; \
+ while (--pos > 0) \
+ (void) va_arg (arg, void *); \
+ va_arg (arg, type); \
+ }))
# endif
#endif
diff --git a/sysdeps/mach/hurd/setitimer.c b/sysdeps/mach/hurd/setitimer.c
index 1e8b40d..61e37c5 100644
--- a/sysdeps/mach/hurd/setitimer.c
+++ b/sysdeps/mach/hurd/setitimer.c
@@ -89,8 +89,8 @@ timer_thread (void)
err = __mach_msg (&msg.header,
MACH_RCV_MSG|MACH_RCV_TIMEOUT|MACH_RCV_INTERRUPT,
0, 0, _hurd_itimer_port,
- _hurd_itimerval.it_value.tv_sec * 1000 +
- _hurd_itimerval.it_value.tv_usec / 1000,
+ _hurd_itimerval.it_value.tv_sec * 1000
+ + _hurd_itimerval.it_value.tv_usec / 1000,
MACH_PORT_NULL);
switch (err)
{
@@ -278,8 +278,8 @@ setitimer_locked (const struct itimerval *new, struct itimerval *old,
kernel context so that when the thread is resumed, mach_msg
will return to timer_thread (below) and it will fetch new
values from _hurd_itimerval. */
- if ((err = __thread_suspend (_hurd_itimer_thread)) ||
- (err = __thread_abort (_hurd_itimer_thread)))
+ if ((err = __thread_suspend (_hurd_itimer_thread))
+ || (err = __thread_abort (_hurd_itimer_thread)))
/* If we can't save it for later, nuke it. */
kill_itimer_thread ();
else
@@ -287,8 +287,8 @@ setitimer_locked (const struct itimerval *new, struct itimerval *old,
}
}
/* See if the timeout changed. If so, we must alert the itimer thread. */
- else if (remaining.tv_sec != newval.it_value.tv_sec ||
- remaining.tv_usec != newval.it_value.tv_usec)
+ else if (remaining.tv_sec != newval.it_value.tv_sec
+ || remaining.tv_usec != newval.it_value.tv_usec)
{
/* The timeout value is changing. Tell the itimer thread to
reexamine it and start counting down. If the itimer thread is
diff --git a/sysdeps/mach/hurd/sigaction.c b/sysdeps/mach/hurd/sigaction.c
index 5e6626d..ae23e22 100644
--- a/sysdeps/mach/hurd/sigaction.c
+++ b/sysdeps/mach/hurd/sigaction.c
@@ -30,10 +30,10 @@ __sigaction (int sig, const struct sigaction *act, struct sigaction *oact)
struct sigaction a, old;
sigset_t pending;
- if (sig <= 0 || sig >= NSIG ||
- (act != NULL && act->sa_handler != SIG_DFL &&
- ((__sigmask (sig) & _SIG_CANT_MASK) ||
- act->sa_handler == SIG_ERR)))
+ if (sig <= 0 || sig >= NSIG
+ || (act != NULL && act->sa_handler != SIG_DFL
+ && ((__sigmask (sig) & _SIG_CANT_MASK)
+ || act->sa_handler == SIG_ERR)))
{
errno = EINVAL;
return -1;
@@ -51,8 +51,8 @@ __sigaction (int sig, const struct sigaction *act, struct sigaction *oact)
if (act != NULL)
ss->actions[sig] = a;
- if (act != NULL && sig == SIGCHLD &&
- (a.sa_flags & SA_NOCLDSTOP) != (old.sa_flags & SA_NOCLDSTOP))
+ if (act != NULL && sig == SIGCHLD
+ && (a.sa_flags & SA_NOCLDSTOP) != (old.sa_flags & SA_NOCLDSTOP))
{
__spin_unlock (&ss->lock);
diff --git a/sysdeps/mach/hurd/sigaltstack.c b/sysdeps/mach/hurd/sigaltstack.c
index 02bdfdd..fc1a9e1 100644
--- a/sysdeps/mach/hurd/sigaltstack.c
+++ b/sysdeps/mach/hurd/sigaltstack.c
@@ -36,8 +36,9 @@ __sigaltstack (const stack_t *argss, stack_t *oss)
s = _hurd_self_sigstate ();
__spin_lock (&s->lock);
- if (argss != NULL &&
- (ss.ss_flags & SS_DISABLE) && (s->sigaltstack.ss_flags & SS_ONSTACK))
+ if (argss != NULL
+ && (ss.ss_flags & SS_DISABLE)
+ && (s->sigaltstack.ss_flags & SS_ONSTACK))
{
/* Can't disable a stack that is in use. */
__spin_unlock (&s->lock);
diff --git a/sysdeps/mach/pagecopy.h b/sysdeps/mach/pagecopy.h
index 6fe5fa3..fff62d3 100644
--- a/sysdeps/mach/pagecopy.h
+++ b/sysdeps/mach/pagecopy.h
@@ -24,9 +24,9 @@
#define PAGE_SIZE __vm_page_size
#define PAGE_COPY_FWD(dstp, srcp, nbytes_left, nbytes) \
- ((nbytes_left) = ((nbytes) - \
- (__vm_copy (__mach_task_self (), \
- (vm_address_t) srcp, trunc_page (nbytes), \
- (vm_address_t) dstp) == KERN_SUCCESS \
- ? trunc_page (nbytes) \
- : 0)))
+ ((nbytes_left) = ((nbytes) \
+ - (__vm_copy (__mach_task_self (), \
+ (vm_address_t) srcp, trunc_page (nbytes), \
+ (vm_address_t) dstp) == KERN_SUCCESS \
+ ? trunc_page (nbytes) \
+ : 0)))
diff --git a/sysdeps/mach/thread_state.h b/sysdeps/mach/thread_state.h
index 2c3bd39..b3977e3 100644
--- a/sysdeps/mach/thread_state.h
+++ b/sysdeps/mach/thread_state.h
@@ -82,8 +82,8 @@ machine_get_basic_state (thread_t thread,
count = MACHINE_THREAD_STATE_COUNT;
if (__thread_get_state (thread, MACHINE_THREAD_STATE_FLAVOR,
(natural_t *) &state->basic,
- &count) != KERN_SUCCESS ||
- count != MACHINE_THREAD_STATE_COUNT)
+ &count) != KERN_SUCCESS
+ || count != MACHINE_THREAD_STATE_COUNT)
/* What kind of thread?? */
return 0; /* XXX */
diff --git a/sysdeps/powerpc/powerpc64/tst-ucontext-ppc64-vscr.c b/sysdeps/powerpc/powerpc64/tst-ucontext-ppc64-vscr.c
index dbe06c8..626edf8 100644
--- a/sysdeps/powerpc/powerpc64/tst-ucontext-ppc64-vscr.c
+++ b/sysdeps/powerpc/powerpc64/tst-ucontext-ppc64-vscr.c
@@ -27,9 +27,9 @@
#define SAT 0x1
/* This test is supported only on POWER 5 or higher. */
-#define PPC_CPU_SUPPORTED (PPC_FEATURE_POWER5 | PPC_FEATURE_POWER5_PLUS | \
- PPC_FEATURE_ARCH_2_05 | PPC_FEATURE_ARCH_2_06 | \
- PPC_FEATURE2_ARCH_2_07)
+#define PPC_CPU_SUPPORTED (PPC_FEATURE_POWER5 | PPC_FEATURE_POWER5_PLUS \
+ | PPC_FEATURE_ARCH_2_05 | PPC_FEATURE_ARCH_2_06 \
+ | PPC_FEATURE2_ARCH_2_07)
static int
do_test (void)
{
diff --git a/sysdeps/unix/sysv/linux/alpha/a.out.h b/sysdeps/unix/sysv/linux/alpha/a.out.h
index 44e19bd..901b914 100644
--- a/sysdeps/unix/sysv/linux/alpha/a.out.h
+++ b/sysdeps/unix/sysv/linux/alpha/a.out.h
@@ -123,9 +123,9 @@ enum machine_type
&& N_MAGIC(x) != ZMAGIC && N_MAGIC(x) != QMAGIC)
#define _N_HDROFF(x) (1024 - sizeof (struct exec))
#define N_TXTOFF(x) \
- ((long) N_MAGIC(x) == ZMAGIC ? 0 : \
- (sizeof (struct exec) + (x).fh.f_nscns * SCNHSZ + SCNROUND - 1) \
- & ~(SCNROUND - 1))
+ ((long) N_MAGIC(x) == ZMAGIC ? 0 \
+ : ((sizeof (struct exec) + (x).fh.f_nscns * SCNHSZ + SCNROUND - 1) \
+ & ~(SCNROUND - 1)))
#define N_DATOFF(x) (N_TXTOFF(x) + (x).a_text)
#define N_TRELOFF(x) (N_DATOFF(x) + (x).a_data)
diff --git a/sysdeps/unix/sysv/linux/generic/wordsize-32/overflow.h b/sysdeps/unix/sysv/linux/generic/wordsize-32/overflow.h
index 105c7d6..6c17047 100644
--- a/sysdeps/unix/sysv/linux/generic/wordsize-32/overflow.h
+++ b/sysdeps/unix/sysv/linux/generic/wordsize-32/overflow.h
@@ -36,8 +36,8 @@ static inline off_t lseek_overflow (loff_t res)
static inline int stat_overflow (struct stat *buf)
{
- if (buf->__st_ino_pad == 0 && buf->__st_size_pad == 0 &&
- buf->__st_blocks_pad == 0)
+ if (buf->__st_ino_pad == 0 && buf->__st_size_pad == 0
+ && buf->__st_blocks_pad == 0)
return 0;
__set_errno (EOVERFLOW);
@@ -47,12 +47,12 @@ static inline int stat_overflow (struct stat *buf)
/* Note that f_files and f_ffree may validly be a sign-extended -1. */
static inline int statfs_overflow (struct statfs *buf)
{
- if (buf->__f_blocks_pad == 0 && buf->__f_bfree_pad == 0 &&
- buf->__f_bavail_pad == 0 &&
- (buf->__f_files_pad == 0 ||
- (buf->f_files == -1U && buf->__f_files_pad == -1)) &&
- (buf->__f_ffree_pad == 0 ||
- (buf->f_ffree == -1U && buf->__f_ffree_pad == -1)))
+ if (buf->__f_blocks_pad == 0 && buf->__f_bfree_pad == 0
+ && buf->__f_bavail_pad == 0
+ && (buf->__f_files_pad == 0
+ || (buf->f_files == -1U && buf->__f_files_pad == -1))
+ && (buf->__f_ffree_pad == 0
+ || (buf->f_ffree == -1U && buf->__f_ffree_pad == -1)))
return 0;
__set_errno (EOVERFLOW);
diff --git a/sysdeps/unix/sysv/linux/tst-personality.c b/sysdeps/unix/sysv/linux/tst-personality.c
index 29be539..881ddfb 100644
--- a/sysdeps/unix/sysv/linux/tst-personality.c
+++ b/sysdeps/unix/sysv/linux/tst-personality.c
@@ -30,11 +30,11 @@ do_test (void)
errno = 0xdefaced;
saved_persona = personality (0xffffffff);
- if (personality (test_persona) != saved_persona ||
- personality (0xffffffff) == -1 ||
- personality (PER_LINUX) == -1 ||
- personality (0xffffffff) != PER_LINUX ||
- 0xdefaced != errno)
+ if (personality (test_persona) != saved_persona
+ || personality (0xffffffff) == -1
+ || personality (PER_LINUX) == -1
+ || personality (0xffffffff) != PER_LINUX
+ || 0xdefaced != errno)
rc = 1;
(void) personality (saved_persona);
diff --git a/sysdeps/unix/sysv/linux/tst-ttyname.c b/sysdeps/unix/sysv/linux/tst-ttyname.c
index 6822b57..872b1fc 100644
--- a/sysdeps/unix/sysv/linux/tst-ttyname.c
+++ b/sysdeps/unix/sysv/linux/tst-ttyname.c
@@ -103,9 +103,9 @@ eq_ttyname (struct result actual, struct result expected)
{
char *actual_name, *expected_name;
- if ((actual.err == expected.err) &&
- (!actual.name == !expected.name) &&
- (actual.name ? strcmp (actual.name, expected.name) == 0 : true))
+ if ((actual.err == expected.err)
+ && (!actual.name == !expected.name)
+ && (actual.name ? strcmp (actual.name, expected.name) == 0 : true))
{
if (expected.name)
expected_name = xasprintf ("\"%s\"", expected.name);
@@ -169,10 +169,10 @@ eq_ttyname_r (struct result_r actual, struct result_r expected)
{
char *actual_name, *expected_name;
- if ((actual.err == expected.err) &&
- (actual.ret == expected.ret) &&
- (!actual.name == !expected.name) &&
- (actual.name ? strcmp (actual.name, expected.name) == 0 : true))
+ if ((actual.err == expected.err)
+ && (actual.ret == expected.ret)
+ && (!actual.name == !expected.name)
+ && (actual.name ? strcmp (actual.name, expected.name) == 0 : true))
{
if (expected.name)
expected_name = xasprintf ("\"%s\"", expected.name);
@@ -570,9 +570,9 @@ run_chroot_tests (const char *slavename, int slave)
struct dirent *d;
while ((d = readdir (dirstream)) != NULL && ci < 3)
{
- if (strcmp (d->d_name, "console1") &&
- strcmp (d->d_name, "console2") &&
- strcmp (d->d_name, "console3") )
+ if (strcmp (d->d_name, "console1")
+ && strcmp (d->d_name, "console2")
+ && strcmp (d->d_name, "console3") )
continue;
c[ci++] = xasprintf ("/dev/%s", d->d_name);
}
-----------------------------------------------------------------------
Summary of changes:
ChangeLog | 21 +++++++++
stdio-common/vfscanf-internal.c | 44 ++++++++++----------
sysdeps/mach/hurd/setitimer.c | 12 +++---
sysdeps/mach/hurd/sigaction.c | 12 +++---
sysdeps/mach/hurd/sigaltstack.c | 5 +-
sysdeps/mach/pagecopy.h | 12 +++---
sysdeps/mach/thread_state.h | 4 +-
.../powerpc/powerpc64/tst-ucontext-ppc64-vscr.c | 6 +-
sysdeps/unix/sysv/linux/alpha/a.out.h | 6 +-
.../unix/sysv/linux/generic/wordsize-32/overflow.h | 16 ++++----
sysdeps/unix/sysv/linux/tst-personality.c | 10 ++--
sysdeps/unix/sysv/linux/tst-ttyname.c | 20 ++++----
12 files changed, 95 insertions(+), 73 deletions(-)
hooks/post-receive
--
GNU C Library master sources