[Beignet] [PATCH] fix typo for DEBUGP to avoid print extra empty line

Guo Yejun yejun.guo at intel.com
Thu Jan 14 14:58:43 PST 2016


for the case without {}:
   if (...)
     DEBUGP(...)

the second printf of DEBUGP is always enabled, fixed it.

Signed-off-by: Guo Yejun <yejun.guo at intel.com>
---
 src/cl_utils.h | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/cl_utils.h b/src/cl_utils.h
index 83f880e..311ba2a 100644
--- a/src/cl_utils.h
+++ b/src/cl_utils.h
@@ -41,9 +41,11 @@ enum DEBUGP_LEVEL
   #define DEBUGP(...)
 #else
   //TODO: decide print or not with the value of level from environment
-  #define DEBUGP(level, fmt, ...)        \
-  fprintf(stderr, "Beignet: "#fmt, ##__VA_ARGS__);  \
-  fprintf(stderr, "\n");
+  #define DEBUGP(level, fmt, ...)                       \
+  do {                                                  \
+    fprintf(stderr, "Beignet: "#fmt, ##__VA_ARGS__);    \
+    fprintf(stderr, "\n");                              \
+  } while (0)
 #endif
 
 /* Check compile time errors */
-- 
1.9.1



More information about the Beignet mailing list