diff options
| author | Joseph Myers <joseph@codesourcery.com> | 2019-02-27 13:58:56 +0000 |
|---|---|---|
| committer | Joseph Myers <joseph@codesourcery.com> | 2019-02-27 13:58:56 +0000 |
| commit | f627dd4da804c00b5e89e5e2c488c2bad6205262 (patch) | |
| tree | 1f7f5ac50d2fda297e69a812e4d05e821856c7f9 | |
| parent | c4f50205e10b066654cf59ecde4fd1c2a674f19e (diff) | |
| download | glibc-f627dd4da804c00b5e89e5e2c488c2bad6205262.tar.xz glibc-f627dd4da804c00b5e89e5e2c488c2bad6205262.zip | |
Fix -Wempty-body warnings in Hurd-specific code.
This patch fixes -Wempty-body warnings in Hurd-specific code that show
up building glibc with -Wextra.
Note: there also such warnings on many platforms arising from the
default definition of HP_TIMING_NOW in sysdeps/generic/hp-timing.h,
but no change there is proposed in this patch because of other changes
under discussion in that area that would result in a nonempty
definition.
Tested with build-many-glibcs.py.
* hurd/hurdinit.c (_hurd_init): Use braces around empty body of an
if statement.
| -rw-r--r-- | ChangeLog | 3 | ||||
| -rw-r--r-- | hurd/hurdinit.c | 10 |
2 files changed, 9 insertions, 4 deletions
@@ -1,5 +1,8 @@ 2019-02-27 Joseph Myers <joseph@codesourcery.com> + * hurd/hurdinit.c (_hurd_init): Use braces around empty body of an + if statement. + * benchtests/bench-strcpy.c (do_test): Use space before '('. * benchtests/bench-string.h (cmdline_process_function): Likewise. * benchtests/bench-strlen.c (do_test): Likewise. diff --git a/hurd/hurdinit.c b/hurd/hurdinit.c index f4a3607ae6..938e6da5e3 100644 --- a/hurd/hurdinit.c +++ b/hurd/hurdinit.c @@ -100,10 +100,12 @@ _hurd_init (int flags, char **argv, portarraysize * sizeof (mach_port_t)); if (flags & EXEC_SECURE) - /* XXX if secure exec, elide environment variables - which the library uses and could be security holes. - CORESERVER, COREFILE - */ ; + { + /* XXX if secure exec, elide environment variables + which the library uses and could be security holes. + CORESERVER, COREFILE + */ + } /* Call other things which want to do some initialization. These are not on the __libc_subinit hook because things there like to be able to |