diff options
author | Treehugger Robot <treehugger-gerrit@google.com> | 2019-11-08 19:48:35 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2019-11-08 19:48:35 +0000 |
commit | 39a5c06ec8b332df2e2ddc53906591697ae7bf28 (patch) | |
tree | 4abbf7898937e05b288f0a19b4af4b288f76a22f | |
parent | 037de111850bb49aa4e92e801ed996f4eecc3fd4 (diff) | |
parent | 1383d9fcd86475fa105c1f1fa75988d5adafad41 (diff) | |
download | native-39a5c06ec8b332df2e2ddc53906591697ae7bf28.tar.gz |
Merge "Replace -Weverything with -Wextra"
-rw-r--r-- | libs/binder/tests/Android.bp | 7 | ||||
-rw-r--r-- | libs/sensor/Android.bp | 17 | ||||
-rw-r--r-- | libs/ui/Android.bp | 23 | ||||
-rw-r--r-- | vulkan/libvulkan/Android.bp | 7 |
4 files changed, 7 insertions, 47 deletions
diff --git a/libs/binder/tests/Android.bp b/libs/binder/tests/Android.bp index 635ea69fb4..5a7f9a97fa 100644 --- a/libs/binder/tests/Android.bp +++ b/libs/binder/tests/Android.bp @@ -119,12 +119,7 @@ cc_test { srcs: ["binderSafeInterfaceTest.cpp"], cppflags: [ - "-Weverything", - "-Wno-c++98-compat", - "-Wno-c++98-compat-pedantic", - "-Wno-global-constructors", - "-Wno-padded", - "-Wno-weak-vtables", + "-Wextra", ], cpp_std: "experimental", diff --git a/libs/sensor/Android.bp b/libs/sensor/Android.bp index 940ff5afbc..e8154a6931 100644 --- a/libs/sensor/Android.bp +++ b/libs/sensor/Android.bp @@ -21,22 +21,7 @@ cc_library_shared { "-Werror", ], cppflags: [ - "-Weverything", - - // The static constructors and destructors in this library have not been noted to - // introduce significant overheads - "-Wno-exit-time-destructors", - "-Wno-global-constructors", - - // We only care about compiling as C++14 - "-Wno-c++98-compat-pedantic", - - // android/sensors.h uses nested anonymous unions and anonymous structs - "-Wno-nested-anon-types", - "-Wno-gnu-anonymous-struct", - - // Don't warn about struct padding - "-Wno-padded", + "-Wextra", ], srcs: [ diff --git a/libs/ui/Android.bp b/libs/ui/Android.bp index 2518e9374d..db15e17524 100644 --- a/libs/ui/Android.bp +++ b/libs/ui/Android.bp @@ -26,28 +26,7 @@ cc_library_shared { "-Werror", ], cppflags: [ - "-Weverything", - - // The static constructors and destructors in this library have not been noted to - // introduce significant overheads - "-Wno-exit-time-destructors", - "-Wno-global-constructors", - - // We only care about compiling as C++14 - "-Wno-c++98-compat-pedantic", - - // We are aware of the risks inherent in comparing floats for equality - "-Wno-float-equal", - - // We use four-character constants for the GraphicBuffer header, and don't care - // that they're non-portable as long as they're consistent within one execution - "-Wno-four-char-constants", - - // Don't warn about struct padding - "-Wno-padded", - - "-Wno-switch-enum", - "-Wno-format-pedantic", + "-Wextra", ], sanitize: { diff --git a/vulkan/libvulkan/Android.bp b/vulkan/libvulkan/Android.bp index b0c4f3f95a..4d6b2be301 100644 --- a/vulkan/libvulkan/Android.bp +++ b/vulkan/libvulkan/Android.bp @@ -41,12 +41,13 @@ cc_library_shared { "-DVK_NO_PROTOTYPES", "-fvisibility=hidden", "-fstrict-aliasing", - "-Weverything", + "-Wextra", "-Werror", "-Wno-padded", + "-Wno-sign-compare", "-Wno-switch-enum", - "-Wno-undef", - "-Wno-format-pedantic", + "-Wno-unused-variable", + "-Wno-unused-function", // Have clang emit complete debug_info. "-fstandalone-debug", |