+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
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
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)
{
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
}
}
/* 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
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;
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);
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);
#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)))
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 */
#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)
{
&& 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)
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);
/* 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);
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);
{
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);
{
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);
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);
}