From f2490d219b04f789fd45750b1c847f97006fac7b Mon Sep 17 00:00:00 2001 From: rofl0r Date: Wed, 29 Nov 2017 03:58:03 +0000 Subject: [PATCH] fix DEBUG macros to be usable like a generic function call the macros were broken for use in a context like if(foo) DEBUG_XXX(); else DEBUG_XXX(); since multiple statements were executed. --- src/pixiewps.h | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/pixiewps.h b/src/pixiewps.h index e2842cd..00aff1d 100644 --- a/src/pixiewps.h +++ b/src/pixiewps.h @@ -45,12 +45,12 @@ #include "utils.h" #if defined(DEBUG) -# define DEBUG_PRINT(fmt, args...); fprintf(stderr, "\n [DEBUG] %s:%d:%s(): " fmt, \ - __FILE__, __LINE__, __func__, ##args); fflush(stdout); -# define DEBUG_PRINT_ARRAY(b, l); byte_array_print(b, l); fflush(stdout); +# define DEBUG_PRINT(fmt, args...) do { fprintf(stderr, "\n [DEBUG] %s:%d:%s(): " fmt, \ + __FILE__, __LINE__, __func__, ##args); fflush(stdout); } while (0) +# define DEBUG_PRINT_ARRAY(b, l) do { byte_array_print(b, l); fflush(stdout); } while (0) #else -# define DEBUG_PRINT(fmt, args...) -# define DEBUG_PRINT_ARRAY(b, l) +# define DEBUG_PRINT(fmt, args...) do {} while (0) +# define DEBUG_PRINT_ARRAY(b, l) do {} while (0) #endif uint_fast8_t p_mode[MODE_LEN] = { 0 };