aboutsummaryrefslogtreecommitdiff
path: root/include
diff options
context:
space:
mode:
authorJoseph Myers <joseph@codesourcery.com>2019-02-13 13:50:13 +0000
committerJoseph Myers <joseph@codesourcery.com>2019-02-13 13:50:13 +0000
commit0538372096e50d9cf0ab8bd413cf7b561a1b85d7 (patch)
tree56006ae16310643ed582e02b7e4a150a8d05693b /include
parent86140c6223b5d14d773cf3050ffd0d14977c2c2d (diff)
downloadglibc-0538372096e50d9cf0ab8bd413cf7b561a1b85d7.zip
glibc-0538372096e50d9cf0ab8bd413cf7b561a1b85d7.tar.gz
glibc-0538372096e50d9cf0ab8bd413cf7b561a1b85d7.tar.bz2
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.
Diffstat (limited to 'include')
-rw-r--r--include/stap-probe.h10
1 files changed, 5 insertions, 5 deletions
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. */