commit 122352d983805b8ded5f8f3c07e75844244aba40
author Chih-Hung Hsieh <chh@google.com> Mon Oct 02 15:20:07 2017 -0700
committer Chih-Hung Hsieh <chh@google.com> Wed Nov 01 11:32:55 2017 -0700
tree 2512a37968404bd39265da943bee33e33e064744
parent 15251c24b9cbb809aad74d8e5796f0eb6d83d8bb

Use -Werror in system/core

* Move -Wall -Werror from cppflags to cflags.
* Fix/suppress warning on unused variables.

Bug: 66996870
Test: build with WITH_TIDY=1
Change-Id: I1e05e96a1d0bcb2ccef1ce456504b3af57167cc5

libutils/Android.bp

diff --git a/libutils/Android.bp b/libutils/Android.bp
index b70845b..4bd2a98 100644
--- a/libutils/Android.bp
+++ b/libutils/Android.bp
@@ -75,7 +75,7 @@
         "misc.cpp",
     ],
 
-    cflags: ["-Werror"],
+    cflags: ["-Wall", "-Werror"],
     include_dirs: ["external/safe-iop/include"],
     header_libs: [
         "libutils_headers",
@@ -154,6 +154,7 @@
     static_libs: ["libutils"],
     shared_libs: ["liblog"],
     srcs: ["SharedBufferTest.cpp"],
+    cflags: ["-Wall", "-Werror"],
 }
 
 subdirs = ["tests"]