diff options
author | Pierre-Clément Tosi <ptosi@google.com> | 2023-04-14 00:55:01 +0000 |
---|---|---|
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2023-04-14 00:55:01 +0000 |
commit | 0930e4cbcaca239a8106d53e92c5c2b312026e48 (patch) | |
tree | cfc3f93f845175781608a930c4b74b578d5e727a | |
parent | f214528a1d58d15aa2c68131ac79dd9bc31793cb (diff) | |
parent | 89064a4e225f8037184d5244aae8389366d0a50b (diff) | |
download | avb-0930e4cbcaca239a8106d53e92c5c2b312026e48.tar.gz |
util: Deprecate avb_{debug,error,fatal}v() macros am: 89064a4e22
Original change: https://android-review.googlesource.com/c/platform/external/avb/+/2520437
Change-Id: Ife5e28dc88fcba69f8a0ed5fea9fd2e498eb750d
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
-rw-r--r-- | libavb/avb_util.h | 21 |
1 files changed, 12 insertions, 9 deletions
diff --git a/libavb/avb_util.h b/libavb/avb_util.h index 09205fc..671d39a 100644 --- a/libavb/avb_util.h +++ b/libavb/avb_util.h @@ -44,7 +44,8 @@ extern "C" { AVB_TO_STRING(__LINE__), \ ": " level ": ", \ message, \ - ##__VA_ARGS__) + ##__VA_ARGS__, \ + NULL) #ifdef AVB_ENABLE_DEBUG /* Aborts the program if |expr| is false. @@ -71,16 +72,14 @@ extern "C" { * * These have no effect unless AVB_ENABLE_DEBUG is defined. */ -#define avb_debug(message) avb_debugv(message, NULL) -#define avb_debugv(message, ...) \ +#define avb_debug(message, ...) \ do { \ AVB_LOG("DEBUG", message, ##__VA_ARGS__); \ } while (0) #else #define avb_assert(expr) #define avb_assert_not_reached() -#define avb_debug(message) -#define avb_debugv(message, ...) +#define avb_debug(message, ...) #endif /* Aborts the program if |addr| is not word-aligned. @@ -93,21 +92,25 @@ extern "C" { /* Prints out a message. This is typically used if a runtime-error * occurs. */ -#define avb_error(message) avb_errorv(message, NULL) -#define avb_errorv(message, ...) \ +#define avb_error(message, ...) \ do { \ AVB_LOG("ERROR", message, ##__VA_ARGS__); \ } while (0) /* Prints out a message and calls avb_abort(). */ -#define avb_fatal(message) avb_fatalv(message, NULL) -#define avb_fatalv(message, ...) \ +#define avb_fatal(message, ...) \ do { \ AVB_LOG("FATAL", message, ##__VA_ARGS__); \ avb_abort(); \ } while (0) +/* Deprecated legacy logging functions -- kept for client compatibility. + */ +#define avb_debugv(message, ...) avb_debug(message, ##__VA_ARGS__) +#define avb_errorv(message, ...) avb_error(message, ##__VA_ARGS__) +#define avb_fatalv(message, ...) avb_fatal(message, ##__VA_ARGS__) + /* Converts a 16-bit unsigned integer from big-endian to host byte order. */ uint16_t avb_be16toh(uint16_t in) AVB_ATTR_WARN_UNUSED_RESULT; |