diff options
author | Chih-Hung Hsieh <chh@google.com> | 2017-10-05 15:20:11 -0700 |
---|---|---|
committer | Bruno Martins <bgcngm@gmail.com> | 2018-04-05 10:30:50 +0100 |
commit | 27615d420e31e608861782a1acc723d1e3189fd6 (patch) | |
tree | f94f630028268cfca19f1e7469688091fafe9c35 | |
parent | 6e58bf095e4f6d5e5123d2f90ea6f2737b6250d3 (diff) | |
download | android_frameworks_native-27615d420e31e608861782a1acc723d1e3189fd6.tar.gz android_frameworks_native-27615d420e31e608861782a1acc723d1e3189fd6.tar.bz2 android_frameworks_native-27615d420e31e608861782a1acc723d1e3189fd6.zip |
Use -Werror in frameworks/native/libs
* Suppress unused-variable and user-defined warnings.
* Keep sign-compare warnings to fix later.
Bug: 66996870
Test: build with WITH_TIDY=1
Change-Id: I6eee1532aa22c92ed19515ad5574734d1016d967
-rw-r--r-- | libs/diskusage/Android.bp | 1 | ||||
-rw-r--r-- | libs/graphicsenv/Android.bp | 2 | ||||
-rw-r--r-- | libs/hwc2on1adapter/Android.bp | 6 | ||||
-rw-r--r-- | libs/input/tests/Android.bp | 8 | ||||
-rw-r--r-- | libs/math/tests/Android.bp | 4 |
5 files changed, 20 insertions, 1 deletions
diff --git a/libs/diskusage/Android.bp b/libs/diskusage/Android.bp index 156ddff2c..a8263069d 100644 --- a/libs/diskusage/Android.bp +++ b/libs/diskusage/Android.bp @@ -15,4 +15,5 @@ cc_library_static { name: "libdiskusage", srcs: ["dirsize.c"], + cflags: ["-Wall", "-Werror"], } diff --git a/libs/graphicsenv/Android.bp b/libs/graphicsenv/Android.bp index f2686d574..9f995380b 100644 --- a/libs/graphicsenv/Android.bp +++ b/libs/graphicsenv/Android.bp @@ -19,6 +19,8 @@ cc_library_shared { "GraphicsEnv.cpp", ], + cflags: ["-Wall", "-Werror"], + shared_libs: [ "libnativeloader", "liblog", diff --git a/libs/hwc2on1adapter/Android.bp b/libs/hwc2on1adapter/Android.bp index ec9cbf842..420a1f606 100644 --- a/libs/hwc2on1adapter/Android.bp +++ b/libs/hwc2on1adapter/Android.bp @@ -17,9 +17,13 @@ cc_library_shared { vendor: true, clang: true, + cflags: [ + "-Wall", + "-Werror", + "-Wno-user-defined-warnings", + ], cppflags: [ "-Weverything", - "-Wall", "-Wunused", "-Wunreachable-code", diff --git a/libs/input/tests/Android.bp b/libs/input/tests/Android.bp index 029a42091..002865507 100644 --- a/libs/input/tests/Android.bp +++ b/libs/input/tests/Android.bp @@ -7,6 +7,12 @@ cc_test { "InputEvent_test.cpp", "InputPublisherAndConsumer_test.cpp", ], + cflags: [ + "-Wall", + "-Werror", + "-Wno-error=sign-compare", // to fix later + "-Wno-unused-variable", + ], shared_libs: [ "libinput", "libcutils", @@ -24,5 +30,7 @@ cc_library_static { srcs: ["StructLayout_test.cpp"], cflags: [ "-O0", + "-Wall", + "-Werror", ], } diff --git a/libs/math/tests/Android.bp b/libs/math/tests/Android.bp index 0ed24a2a1..0184f56dc 100644 --- a/libs/math/tests/Android.bp +++ b/libs/math/tests/Android.bp @@ -18,22 +18,26 @@ cc_test { name: "vec_test", srcs: ["vec_test.cpp"], static_libs: ["libmath"], + cflags: ["-Wall", "-Werror"], } cc_test { name: "mat_test", srcs: ["mat_test.cpp"], static_libs: ["libmath"], + cflags: ["-Wall", "-Werror"], } cc_test { name: "half_test", srcs: ["half_test.cpp"], static_libs: ["libmath"], + cflags: ["-Wall", "-Werror"], } cc_test { name: "quat_test", srcs: ["quat_test.cpp"], static_libs: ["libmath"], + cflags: ["-Wall", "-Werror"], } |