Silence unused variable warnings caused by BT_ASSERT() in non-debug mode
authorFrancis Deslauriers <francis.deslauriers@efficios.com>
Tue, 3 Apr 2018 20:24:49 +0000 (16:24 -0400)
committerPhilippe Proulx <eeppeliteloop@gmail.com>
Thu, 2 May 2019 04:05:45 +0000 (00:05 -0400)
commitd7da1f66c46f7fa93ec4d70a889f9274e06f472e
tree1c6656bd10a6698904dda12eed9cf7e7cc57f5a5
parent2d445b065690292239783d28735e99cd79def10d
Silence unused variable warnings caused by BT_ASSERT() in non-debug mode

When building with the BABELTRACE_DEBUG_MODE configure option _not_ set,
multiple set-but-not-used warnings are emitted by the compiler. Most of
the time this happens because BT_ASSERT() is used to verify the return
value of functions and when not in DEBUG_MODE those return value are not
used.

The solution is to explicitly tell the compiler that we don't care about
this value so that it does not emit warning when building in non-debug
mode. This approach also makes sure not to evaluate the expression using
the `sizeof` operator, thus preventing any side effects.

See this post for further detailsĀ [1].

[1]: https://stackoverflow.com/questions/37411809/how-to-elegantly-fix-this-unused-variable-warning/37412551#37412551

Signed-off-by: Francis Deslauriers <francis.deslauriers@efficios.com>
include/babeltrace/assert-internal.h
This page took 0.02606 seconds and 4 git commands to generate.