aboutsummaryrefslogtreecommitdiffstats
path: root/include/osmocom/core/utils.h
diff options
context:
space:
mode:
authorAlexander Couzens <lynxis@fe80.eu>2019-06-23 01:53:43 +0200
committerlynxis lazus <lynxis@fe80.eu>2019-07-08 13:23:45 +0000
commit4e284b637943980a405a8c44f2712b749ded428f (patch)
tree2658472dcbe3646f451623fca8c928e004ae0546 /include/osmocom/core/utils.h
parent316d1e1b7bbe4c6d1c9b6adbd27ecba3b20f3743 (diff)
utils.h: require a semi colon after OSMO_ASSERT
When using `OSMO_ASSERT(exp);` clang will warn about an empty expression because the semi colon was superflous. Use do {} while (0) to enfore the need of a semi colon. This might break other test. Change-Id: I2272d29a81496164bebd1696a694383a28a86434
Diffstat (limited to 'include/osmocom/core/utils.h')
-rw-r--r--include/osmocom/core/utils.h4
1 files changed, 2 insertions, 2 deletions
diff --git a/include/osmocom/core/utils.h b/include/osmocom/core/utils.h
index 601bb565..8585a6a9 100644
--- a/include/osmocom/core/utils.h
+++ b/include/osmocom/core/utils.h
@@ -98,9 +98,9 @@ do { \
* the predicate evaluates to false (0).
*/
#define OSMO_ASSERT(exp) \
- if (!(exp)) { \
+ do if (!(exp)) { \
osmo_panic("Assert failed %s %s:%d\n", #exp, __FILE__, __LINE__); \
- }
+ } while(0)
/*! duplicate a string using talloc and release its prior content (if any)
* \param[in] ctx Talloc context to use for allocation