From c17a4dc34c0aa449e843595319fd9380c2a753b4 Mon Sep 17 00:00:00 2001 From: Sandrine Bailleux Date: Wed, 14 Oct 2015 16:00:23 +0100 Subject: [PATCH] Make CASSERT() macro callable from anywhere The CASSERT() macro introduces a typedef for the sole purpose of triggering a compilation error if the condition to check is false. This typedef is not used afterwards. As a consequence, when the CASSERT() macro is called from withing a function block, the compiler complains and outputs the following error message: error: typedef 'msg' locally defined but not used [-Werror=unused-local-typedefs] This patch adds the "unused" attribute for the aforementioned typedef. This silences the compiler warning and thus makes the CASSERT() macro callable from within function blocks as well. Change-Id: Ie36b58fcddae01a21584c48bb6ef43ec85590479 --- include/lib/cassert.h | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/include/lib/cassert.h b/include/lib/cassert.h index 0e5529dd6..e8089cbbe 100644 --- a/include/lib/cassert.h +++ b/include/lib/cassert.h @@ -34,9 +34,12 @@ /******************************************************************************* * Macro to flag a compile time assertion. It uses the preprocessor to generate * an invalid C construct if 'cond' evaluates to false. - * The following compilation error is triggered if the assertion fails: + * The following compilation error is triggered if the assertion fails: * "error: size of array 'msg' is negative" + * The 'unused' attribute ensures that the unused typedef does not emit a + * compiler warning. ******************************************************************************/ -#define CASSERT(cond, msg) typedef char msg[(cond) ? 1 : -1] +#define CASSERT(cond, msg) \ + typedef char msg[(cond) ? 1 : -1] __attribute__((unused)) #endif /* __CASSERT_H__ */