espmissingincludes.h

encapsulate os_printf macro with do/while to avoid unwanted side effects during macro expansion
9 years ago · 30ff23c0e2
parent 538e79f1dc
commit 30ff23c0e2
1 changed files with 5 additions and 4 deletions
--- a/include/espmissingincludes.h
+++ b/include/espmissingincludes.h
@ -53,10 +53,11 @@ int os_snprintf(char *str, size_t size, const char *format, ...) __attribute__((
 int os_printf_plus(const char *format, ...)  __attribute__((format(printf, 1, 2)));

 #undef os_printf
-#define os_printf(format, ...) \
-  system_set_os_print(true); \
-  os_printf_plus(format, ## __VA_ARGS__); \
-  system_set_os_print(DEBUG_SDK); // int os_printf(const char *format, ...)
+#define os_printf(format, ...) do {                                           \
+    system_set_os_print(true);                                                \
+    os_printf_plus(format, ## __VA_ARGS__);                                   \
+    system_set_os_print(DEBUG_SDK);                                           \
+  } while (0)


 // memory allocation functions are "different" due to memory debugging functionality