| Message ID | 1455048108-5045-2-git-send-email-andrew.cooper3@citrix.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
diff --git a/xen/include/xen/lib.h b/xen/include/xen/lib.h index 4258912..76232b2 100644 --- a/xen/include/xen/lib.h +++ b/xen/include/xen/lib.h @@ -32,7 +32,7 @@ #define ASSERT_UNREACHABLE() assert_failed("unreachable") #define debug_build() 1 #else -#define ASSERT(p) do { if ( 0 && (p) ); } while (0) +#define ASSERT(p) do { if ( 0 && (p) ){} } while (0) #define ASSERT_UNREACHABLE() do { } while (0) #define debug_build() 0 #endif
Clang warns about a semicolon immediately following an if() clause as a possible mistake, and recommends putting the semicolon on a new line if it was intentional. A newline is not an option here, so use a set of empty braces instead. Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> --- CC: Jan Beulich <JBeulich@suse.com> CC: Tim Deegan <tim@xen.org> CC: Ian Campbell <Ian.Campbell@citrix.com> --- xen/include/xen/lib.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)