AudioFlinger: Group common compile flags together
Test: compiles
Bug: 286936439
Change-Id: Ie15e07886e2e4fcc813b1cea18ad87bc1b892fd4
diff --git a/services/audioflinger/Android.bp b/services/audioflinger/Android.bp
index 4ebfa10..fec327f 100644
--- a/services/audioflinger/Android.bp
+++ b/services/audioflinger/Android.bp
@@ -97,37 +97,38 @@
"-performance-no-int-to-ptr",
]
+audioflinger_base_cflags = [
+ "-Wall",
+ "-Wdeprecated",
+ "-Werror",
+ "-Werror=implicit-fallthrough",
+ "-Werror=sometimes-uninitialized",
+ "-Werror=conditional-uninitialized",
+ "-Wextra",
+
+ // suppress some warning chatter.
+ "-Wno-deprecated-copy-with-dtor",
+ "-Wno-deprecated-copy-with-user-provided-dtor",
+
+ "-Wredundant-decls",
+ "-Wshadow",
+ "-Wstrict-aliasing",
+ "-fstrict-aliasing",
+ "-Wthread-safety",
+ //"-Wthread-safety-negative", // experimental - looks broken in R.
+ "-Wunreachable-code",
+ "-Wunreachable-code-break",
+ "-Wunreachable-code-return",
+ "-Wunused",
+ "-Wused-but-marked-unused",
+]
+
// Eventually use common tidy defaults
cc_defaults {
name: "audioflinger_flags_defaults",
// https://clang.llvm.org/docs/UsersManual.html#command-line-options
// https://clang.llvm.org/docs/DiagnosticsReference.html
- cflags: [
- "-Wall",
- "-Wdeprecated",
- "-Werror",
- "-Werror=implicit-fallthrough",
- "-Werror=sometimes-uninitialized",
- "-Werror=conditional-uninitialized",
- "-Wextra",
-
- // suppress some warning chatter.
- "-Wno-deprecated-copy-with-dtor",
- "-Wno-deprecated-copy-with-user-provided-dtor",
-
- "-Wredundant-decls",
- "-Wshadow",
- "-Wstrict-aliasing",
- "-fstrict-aliasing",
- "-Wthread-safety",
- //"-Wthread-safety-negative", // experimental - looks broken in R.
- "-Wunreachable-code",
- "-Wunreachable-code-break",
- "-Wunreachable-code-return",
- "-Wunused",
- "-Wused-but-marked-unused",
- "-D_LIBCPP_ENABLE_THREAD_SAFETY_ANNOTATIONS",
- ],
+ cflags: audioflinger_base_cflags,
// https://clang.llvm.org/extra/clang-tidy/
tidy: true,
tidy_checks: audioflinger_tidy_errors,