This is the mail archive of the glibc-cvs@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

GNU C Library master sources branch master updated. glibc-2.29.9000-96-gf627dd4


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  f627dd4da804c00b5e89e5e2c488c2bad6205262 (commit)
      from  c4f50205e10b066654cf59ecde4fd1c2a674f19e (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=f627dd4da804c00b5e89e5e2c488c2bad6205262

commit f627dd4da804c00b5e89e5e2c488c2bad6205262
Author: Joseph Myers <joseph@codesourcery.com>
Date:   Wed Feb 27 13:58:56 2019 +0000

    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.

diff --git a/ChangeLog b/ChangeLog
index 34e4558..de34cdd 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -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 f4a3607..938e6da 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

-----------------------------------------------------------------------

Summary of changes:
 ChangeLog       |    3 +++
 hurd/hurdinit.c |   10 ++++++----
 2 files changed, 9 insertions(+), 4 deletions(-)


hooks/post-receive
-- 
GNU C Library master sources


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]