[PATCH:makedepend] Use do { ... } while(0) idiom to make debug() macro safer
Alan Coopersmith
alan.coopersmith at oracle.com
Sat Mar 12 18:15:15 UTC 2016
Cleans up several -Wempty-body warnings from gcc 5.3
Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
---
def.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/def.h b/def.h
index 1930cde..59670a9 100644
--- a/def.h
+++ b/def.h
@@ -82,9 +82,9 @@ extern int _debugmask;
* 3 show #include SYMBOL
* 4-6 unused
*/
-#define debug(level,arg) { if (_debugmask & (1 << level)) warning arg; }
+#define debug(level,arg) do { if (_debugmask & (1 << level)) warning arg; } while(0)
#else
-#define debug(level,arg) /**/
+#define debug(level,arg) do { /**/ } while (0)
#endif /* DEBUG */
typedef unsigned char boolean;
--
2.6.1
More information about the xorg-devel
mailing list