From bf8d3d6aca3f20255a621ed1c148fd05b3a8ae5c Mon Sep 17 00:00:00 2001 From: David Lamparter Date: Sun, 21 Feb 2021 06:18:10 +0100 Subject: *: require semicolon after DEFINE_MTYPE & co Back when I put this together in 2015, ISO C11 was still reasonably new and we couldn't require it just yet. Without ISO C11, there is no "good" way (only bad hacks) to require a semicolon after a macro that ends with a function definition. And if you added one anyway, you'd get "spurious semicolon" warnings on some compilers... With C11, `_Static_assert()` at the end of a macro will make it so that the semicolon is properly required, consumed, and not warned about. Consistently requiring semicolons after "file-level" macros matches Linux kernel coding style and helps some editors against mis-syntax'ing these macros. Signed-off-by: David Lamparter --- zebra/zebra_memory.h | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'zebra/zebra_memory.h') diff --git a/zebra/zebra_memory.h b/zebra/zebra_memory.h index 71901b765..769a808cb 100644 --- a/zebra/zebra_memory.h +++ b/zebra/zebra_memory.h @@ -28,11 +28,11 @@ extern "C" { #endif -DECLARE_MGROUP(ZEBRA) -DECLARE_MTYPE(ZEBRA_NS) -DECLARE_MTYPE(RE) -DECLARE_MTYPE(RIB_DEST) -DECLARE_MTYPE(OPAQUE) +DECLARE_MGROUP(ZEBRA); +DECLARE_MTYPE(ZEBRA_NS); +DECLARE_MTYPE(RE); +DECLARE_MTYPE(RIB_DEST); +DECLARE_MTYPE(OPAQUE); #ifdef __cplusplus } -- cgit v1.2.3