From 9049dfdb6895739c3ea1b110e41548a9677194d4 Mon Sep 17 00:00:00 2001 From: Fred Reitberger Date: Wed, 12 Oct 2022 10:47:39 -0400 Subject: util/cbfstool: Wrap logging macros in do - while Wrap the console logging macros with do { ... } while (0) so they act more like functions. Add missing semicolons to calls of these macros. TEST=compile only Signed-off-by: Fred Reitberger Change-Id: I721a4a93636201fa2394ec62cbe4e743cd3ad9d0 Reviewed-on: https://review.coreboot.org/c/coreboot/+/68336 Tested-by: build bot (Jenkins) Reviewed-by: Julius Werner --- util/cbfstool/console/console.h | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) (limited to 'util/cbfstool/console/console.h') diff --git a/util/cbfstool/console/console.h b/util/cbfstool/console/console.h index 0595bea5cc..b25589ad26 100644 --- a/util/cbfstool/console/console.h +++ b/util/cbfstool/console/console.h @@ -8,15 +8,15 @@ /* Message output */ extern int verbose; -#define ERROR(...) { fprintf(stderr, "E: " __VA_ARGS__); } -#define WARN(...) { fprintf(stderr, "W: " __VA_ARGS__); } -#define LOG(...) { fprintf(stderr, __VA_ARGS__); } -#define INFO(...) { if (verbose > 0) fprintf(stderr, "INFO: " __VA_ARGS__); } -#define DEBUG(...) { if (verbose > 1) fprintf(stderr, "DEBUG: " __VA_ARGS__); } +#define ERROR(...) fprintf(stderr, "E: " __VA_ARGS__) +#define WARN(...) fprintf(stderr, "W: " __VA_ARGS__) +#define LOG(...) fprintf(stderr, __VA_ARGS__) +#define INFO(...) do { if (verbose > 0) fprintf(stderr, "INFO: " __VA_ARGS__); } while (0) +#define DEBUG(...) do { if (verbose > 1) fprintf(stderr, "DEBUG: " __VA_ARGS__); } while (0) #define printk(lvl, ...) \ - { \ + do { \ if ((lvl) <= BIOS_ERR) { \ ERROR(__VA_ARGS__); \ } else if ((lvl) <= BIOS_NOTICE) { \ @@ -26,6 +26,6 @@ extern int verbose; } else if ((lvl) <= BIOS_DEBUG) { \ DEBUG(__VA_ARGS__); \ } \ - } + } while (0) #endif -- cgit v1.2.3