diff options
author | Pierre-Clément Tosi <ptosi@google.com> | 2023-04-14 01:54:45 +0000 |
---|---|---|
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2023-04-14 01:54:45 +0000 |
commit | 0546b66f38fa46fca4ec760564502edb9158ac1f (patch) | |
tree | cfc3f93f845175781608a930c4b74b578d5e727a | |
parent | 999a5cf4235c1ede727057295f1a4a86af78824d (diff) | |
parent | 249b65a90e5103e36c738bf411b72411662f3286 (diff) | |
download | avb-0546b66f38fa46fca4ec760564502edb9158ac1f.tar.gz |
util: Deprecate avb_{debug,error,fatal}v() macros am: 89064a4e22 am: 0930e4cbca am: 249b65a90e
Original change: https://android-review.googlesource.com/c/platform/external/avb/+/2520437
Change-Id: Ic5779e62538cfc14ac437188c0b52494bfd15feb
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; |