Replace -Weverything with -Wextra
-Weverything is not recommended, use -Wextra.
Requires adding a few more disabled warnings to libvulkan,
-Weverything appears to not turn on warnings that were disabled
by earlier command line arguments, while -Wextra does.
Bug: 143713277
Test: m native
Change-Id: I3997c2afe094deca66018e3cc43921aaecd82028
diff --git a/libs/ui/Android.bp b/libs/ui/Android.bp
index 2518e93..db15e17 100644
--- a/libs/ui/Android.bp
+++ b/libs/ui/Android.bp
@@ -26,28 +26,7 @@
"-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: {