about summary refs log tree commit diff
path: root/hurd
diff options
context:
space:
mode:
authorJoseph Myers <joseph@codesourcery.com>2019-02-27 13:58:56 +0000
committerJoseph Myers <joseph@codesourcery.com>2019-02-27 13:58:56 +0000
commitf627dd4da804c00b5e89e5e2c488c2bad6205262 (patch)
tree1f7f5ac50d2fda297e69a812e4d05e821856c7f9 /hurd
parentc4f50205e10b066654cf59ecde4fd1c2a674f19e (diff)
downloadglibc-f627dd4da804c00b5e89e5e2c488c2bad6205262.tar.gz
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.
Diffstat (limited to 'hurd')
-rw-r--r--hurd/hurdinit.c10
1 files changed, 6 insertions, 4 deletions
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