for the case without {}:
if (...)
DEBUGP(...)
the second printf of DEBUGP is always enabled, fixed it.
Signed-off-by: Guo Yejun <yejun....@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
_______________________________________________
Beignet mailing list
Beignet@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/beignet
