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