[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