summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlan Coopersmith <alan.coopersmith@oracle.com>2016-03-12 10:11:03 -0800
committerAlan Coopersmith <alan.coopersmith@oracle.com>2016-03-27 10:42:24 -0700
commita4f82f917a9b0802b70cd80c21d2e53688e2b9b3 (patch)
treefe4072c7c540ff17d345ed2ef2782316ee7e4821
parent9da873c5a036c95aa24a7dc3562d405609262381 (diff)
Use do { ... } while(0) idiom to make debug() macro safer
Cleans up several -Wempty-body warnings from gcc 5.3 Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Adam Jackson <ajax@redhat.com>
-rw-r--r--def.h4
1 files 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;