lib: add internal object pool API and use it; adapt plugins/tests

[babeltrace.git] / include / babeltrace / assert-internal.h

diff --git a/include/babeltrace/assert-internal.h b/include/babeltrace/assert-internal.h
index c8a9f8377b469b48131cbe41e875d982dd101724..431230c91d9f145e1e9f0f86ef99cdaef963eab9 100644 (file)
--- a/include/babeltrace/assert-internal.h
+++ b/include/babeltrace/assert-internal.h
@@ -25,6 +25,7 @@
  */
 
 #include <assert.h>
+#include <babeltrace/babeltrace-internal.h>
 
 #ifdef BT_DEBUG_MODE
 /*
@@ -39,8 +40,18 @@
  */
 # define BT_ASSERT_FUNC
 #else
-# define BT_ASSERT(_cond)
-# define BT_ASSERT_FUNC                __attribute__((unused))
+/*
+ * When BT_DEBUG_MODE is not defined, define BT_ASSERT() macro to the following
+ * to trick the compiler into thinking that the variable passed as condition to
+ * the assertion is used. This is to prevent set-but-not-used warnings from the
+ * compiler when assertions are disabled. The `sizeof` operator also makes sure
+ * that the `_cond` expression is not evaluated, thus preventing unwanted side
+ * effects.
+ *
+ * In-depth explanation: https://stackoverflow.com/questions/37411809/how-to-elegantly-fix-this-unused-variable-warning/37412551#37412551
+ */
+# define BT_ASSERT(_cond)      ((void) sizeof((void) (_cond), 0))
+# define BT_ASSERT_FUNC                BT_UNUSED
 #endif /* BT_DEBUG_MODE */
 
 #endif /* BABELTRACE_ASSERT_INTERNAL_H */