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-58-g0538372
- From: jsm28 at sourceware dot org
- To: glibc-cvs at sourceware dot org
- Date: 13 Feb 2019 13:51:02 -0000
- Subject: GNU C Library master sources branch master updated. glibc-2.29.9000-58-g0538372
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 0538372096e50d9cf0ab8bd413cf7b561a1b85d7 (commit)
from 86140c6223b5d14d773cf3050ffd0d14977c2c2d (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=0538372096e50d9cf0ab8bd413cf7b561a1b85d7
commit 0538372096e50d9cf0ab8bd413cf7b561a1b85d7
Author: Joseph Myers <joseph@codesourcery.com>
Date: Wed Feb 13 13:50:13 2019 +0000
Fix -Wempty-body warnings in glibc.
One group of warnings seen building glibc with -Wextra is -Wempty-body
warnings about an 'if' body (or in one case an 'else' body) that is
just a semicolon, "warning: suggest braces around empty body in an
'if' statement [-Wempty-body]" - I think the point of the warning
being to make it more visible whether an 'if' body is actually present
or not.
This patch fixes such warnings in glibc. There's one place, with a
semicolon at the end of a comment, where this is clearly making the
presence of an 'else' body more visible. The other cases involve
macro definitions expanding to nothing. While there's no issue there
with visibility at the call sites, I think it's still cleaner to have
a macro that expands to something nonempty appropriate for the context
- so do {} while (0) if it's only intended to be usable as a
statement, or ((void) 0) where the macro definition is an alternative
to a call to a function returning void, so this patch makes those
changes.
Tested for x86_64.
* catgets/gencat.c (normalize_line): Use braces around empty
'else' body.
* include/stap-probe.h [!USE_STAP_PROBE && !__ASSEMBLER__]
(STAP_PROBE0): Use do {} while (0) for do-nothing definition.
[!USE_STAP_PROBE && !__ASSEMBLER__] (STAP_PROBE1): Likewise.
[!USE_STAP_PROBE && !__ASSEMBLER__] (STAP_PROBE2): Likewise.
[!USE_STAP_PROBE && !__ASSEMBLER__] (STAP_PROBE3): Likewise.
[!USE_STAP_PROBE && !__ASSEMBLER__] (STAP_PROBE4): Likewise.
* libio/libio.h (_IO_funlockfile): Use ((void) 0) for do-nothing
definition.
diff --git a/ChangeLog b/ChangeLog
index 4279eee..f8210cd 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,16 @@
2019-02-13 Joseph Myers <joseph@codesourcery.com>
+ * catgets/gencat.c (normalize_line): Use braces around empty
+ 'else' body.
+ * include/stap-probe.h [!USE_STAP_PROBE && !__ASSEMBLER__]
+ (STAP_PROBE0): Use do {} while (0) for do-nothing definition.
+ [!USE_STAP_PROBE && !__ASSEMBLER__] (STAP_PROBE1): Likewise.
+ [!USE_STAP_PROBE && !__ASSEMBLER__] (STAP_PROBE2): Likewise.
+ [!USE_STAP_PROBE && !__ASSEMBLER__] (STAP_PROBE3): Likewise.
+ [!USE_STAP_PROBE && !__ASSEMBLER__] (STAP_PROBE4): Likewise.
+ * libio/libio.h (_IO_funlockfile): Use ((void) 0) for do-nothing
+ definition.
+
* support/test-container.c (recursive_remove): Use FAIL_EXIT1 if
execlp returns.
diff --git a/catgets/gencat.c b/catgets/gencat.c
index 9d6db70..51fd2a7 100644
--- a/catgets/gencat.c
+++ b/catgets/gencat.c
@@ -1196,7 +1196,9 @@ normalize_line (const char *fname, size_t line, iconv_t cd, wchar_t *string,
++rp;
}
else
- /* Simply ignore the backslash character. */;
+ {
+ /* Simply ignore the backslash character. */
+ }
break;
}
}
diff --git a/include/stap-probe.h b/include/stap-probe.h
index 8c26292..85f41c9 100644
--- a/include/stap-probe.h
+++ b/include/stap-probe.h
@@ -60,11 +60,11 @@
/* Evaluate all the arguments and verify that N matches their number. */
# define LIBC_PROBE(name, n, ...) STAP_PROBE##n (__VA_ARGS__)
-# define STAP_PROBE0()
-# define STAP_PROBE1(a1)
-# define STAP_PROBE2(a1, a2)
-# define STAP_PROBE3(a1, a2, a3)
-# define STAP_PROBE4(a1, a2, a3, a4)
+# define STAP_PROBE0() do {} while (0)
+# define STAP_PROBE1(a1) do {} while (0)
+# define STAP_PROBE2(a1, a2) do {} while (0)
+# define STAP_PROBE3(a1, a2, a3) do {} while (0)
+# define STAP_PROBE4(a1, a2, a3, a4) do {} while (0)
# else
# define LIBC_PROBE(name, n, ...) /* Nothing. */
diff --git a/libio/libio.h b/libio/libio.h
index d21162a..9c08c7e 100644
--- a/libio/libio.h
+++ b/libio/libio.h
@@ -240,7 +240,7 @@ extern int _IO_ftrylockfile (FILE *) __THROW;
#define _IO_peekc(_fp) _IO_peekc_unlocked (_fp)
#define _IO_flockfile(_fp) /**/
-#define _IO_funlockfile(_fp) /**/
+#define _IO_funlockfile(_fp) ((void) 0)
#define _IO_ftrylockfile(_fp) /**/
#ifndef _IO_cleanup_region_start
#define _IO_cleanup_region_start(_fct, _fp) /**/
-----------------------------------------------------------------------
Summary of changes:
ChangeLog | 11 +++++++++++
catgets/gencat.c | 4 +++-
include/stap-probe.h | 10 +++++-----
libio/libio.h | 2 +-
4 files changed, 20 insertions(+), 7 deletions(-)
hooks/post-receive
--
GNU C Library master sources