commit b0e17c9e91bd51a129933e46ef1b68e57bb4b5f3
author Krzysztof Wesolowski <krzysztof.wesolowski@volvocars.com> Thu Nov 30 10:11:13 2017 +0100
committer Bryan Eyler <bryaneyler@google.com> Thu Jan 11 11:11:20 2018 -0800
tree 7a0b738df8a34e847d16ceb26bbe253749da6af3
parent 4d281880e0f6f8644c530e5274f999bebc19d65c

Improve ALOGV compatiblity with clang-tidy

Currently used clang tidy does not filter the warnings caused
by macros, even if macros come from filtered headers.

This change allows projects using ALOGV to use
readability-implicit-bool-conversion without spreading //NOLINT everywhere.

Bug: 71533509
Change-Id: Id1b193d1e56d13f00171e04f600292877c5f6cb3

liblog/include/log/log_main.h

diff --git a/liblog/include/log/log_main.h b/liblog/include/log/log_main.h
index 339a06d..21fc7cc 100644
--- a/liblog/include/log/log_main.h
+++ b/liblog/include/log/log_main.h
@@ -17,6 +17,8 @@
 #ifndef _LIBS_LOG_LOG_MAIN_H
 #define _LIBS_LOG_LOG_MAIN_H
 
+#include <stdbool.h>
+
 #include <android/log.h>
 #include <sys/cdefs.h>
 
@@ -175,10 +177,10 @@
 #if LOG_NDEBUG
 #define ALOGV(...)          \
   do {                      \
-    if (0) {                \
+    if (false) {            \
       __ALOGV(__VA_ARGS__); \
     }                       \
-  } while (0)
+  } while (false)
 #else
 #define ALOGV(...) __ALOGV(__VA_ARGS__)
 #endif