mirror of
https://github.com/chararomandroid/android_device_samsung_a20
synced 2026-08-21 11:55:31 -04:00
universal7885: Remove eureka_defaults treewide
clang-tidy checks doesn't make it best code
This commit is contained in:
roynatech2544
parent
a06a00d26b
commit
092cf54ced
13 changed files with 1 additions and 85 deletions
69
Android.bp
69
Android.bp
|
|
@ -1,69 +0,0 @@
|
|||
// From system/apex/apexd/Android.bp
|
||||
|
||||
tidy_errors = [
|
||||
"android-*",
|
||||
"bugprone-infinite-loop",
|
||||
"bugprone-macro-parentheses",
|
||||
"bugprone-misplaced-widening-cast",
|
||||
"bugprone-move-forwarding-reference",
|
||||
"bugprone-sizeof-container",
|
||||
"bugprone-sizeof-expression",
|
||||
"bugprone-string-constructor",
|
||||
"bugprone-terminating-continue",
|
||||
"bugprone-undefined-memory-manipulation",
|
||||
"bugprone-undelegated-constructor",
|
||||
"bugprone-unhandled-self-assignment",
|
||||
"bugprone-unused-raii",
|
||||
"cert-err34-c",
|
||||
"google-default-arguments",
|
||||
"google-explicit-constructor",
|
||||
"google-readability-avoid-underscore-in-googletest-name",
|
||||
"google-readability-todo",
|
||||
"google-runtime-int",
|
||||
"google-runtime-member-string-references",
|
||||
"misc-move-const-arg",
|
||||
"misc-move-forwarding-reference",
|
||||
"misc-unused-parameters",
|
||||
"misc-unused-using-decls",
|
||||
"misc-use-after-move",
|
||||
"modernize-pass-by-value",
|
||||
"performance-faster-string-find",
|
||||
"performance-for-range-copy",
|
||||
"performance-implicit-conversion-in-loop",
|
||||
"performance-inefficient-vector-operation",
|
||||
"performance-move-const-arg",
|
||||
"performance-move-constructor-init",
|
||||
"performance-noexcept-move-constructor",
|
||||
"performance-unnecessary-copy-initialization",
|
||||
"readability-avoid-const-params-in-decls",
|
||||
]
|
||||
|
||||
cc_defaults {
|
||||
name: "eureka_defaults",
|
||||
cflags: [
|
||||
"-Wall",
|
||||
"-Wextra",
|
||||
"-Werror",
|
||||
"-Wno-unused-parameter",
|
||||
|
||||
// Some extra flags.
|
||||
"-fstrict-aliasing",
|
||||
"-Wredundant-decls",
|
||||
"-Wshadow",
|
||||
"-Wstrict-aliasing",
|
||||
"-Wthread-safety",
|
||||
"-Wunreachable-code",
|
||||
"-Wunreachable-code-break",
|
||||
"-Wunreachable-code-return",
|
||||
"-Wunused",
|
||||
"-Wused-but-marked-unused",
|
||||
],
|
||||
tidy: true,
|
||||
tidy_checks: tidy_errors,
|
||||
tidy_checks_as_errors: tidy_errors,
|
||||
tidy_flags: [
|
||||
"-format-style=file",
|
||||
],
|
||||
}
|
||||
|
||||
|
||||
|
|
@ -2,7 +2,6 @@ cc_binary {
|
|||
name: "android.hardware.sensors@1.0-service.a10",
|
||||
relative_install_path: "hw",
|
||||
vendor: true,
|
||||
defaults: ["eureka_defaults"],
|
||||
init_rc: ["android.hardware.sensors@1.0-service.a10.rc"],
|
||||
srcs: ["service.cpp"],
|
||||
compile_multilib: "32",
|
||||
|
|
|
|||
|
|
@ -18,7 +18,6 @@ cc_binary {
|
|||
relative_install_path: "hw",
|
||||
init_rc: ["android.hardware.usb@1.0-service.a10.rc"],
|
||||
vintf_fragments: ["android.hardware.usb@1.0-service.a10.xml"],
|
||||
defaults: ["eureka_defaults"],
|
||||
srcs: [
|
||||
"service.cpp",
|
||||
"Usb.cpp",
|
||||
|
|
|
|||
|
|
@ -17,8 +17,7 @@ cc_binary {
|
|||
name: "android.hardware.usb@1.0-service.a10dd",
|
||||
relative_install_path: "hw",
|
||||
init_rc: ["android.hardware.usb@1.0-service.a10dd.rc"],
|
||||
vintf_fragments: ["android.hardware.usb@1.0-service.a10dd.xml"],
|
||||
defaults: ["eureka_defaults"],
|
||||
vintf_fragments: ["android.hardware.usb@1.0-service.a10dd.xml"]
|
||||
srcs: [
|
||||
"service.cpp",
|
||||
"Usb.cpp",
|
||||
|
|
|
|||
|
|
@ -1,6 +1,5 @@
|
|||
cc_library_shared {
|
||||
name: "libStorageHelper",
|
||||
defaults: ["eureka_defaults"],
|
||||
cflags: [
|
||||
"-Wall",
|
||||
"-Wextra",
|
||||
|
|
@ -14,7 +13,6 @@ cc_library_shared {
|
|||
|
||||
cc_library_shared {
|
||||
name: "libSwapCallback",
|
||||
defaults: ["eureka_defaults"],
|
||||
srcs: [
|
||||
"SwapCallbackImpl.cpp",
|
||||
"SwapCallbackNative.cpp",
|
||||
|
|
|
|||
|
|
@ -4,9 +4,6 @@ cc_binary {
|
|||
"AudioRoute.cpp",
|
||||
"service.cpp",
|
||||
],
|
||||
defaults: [
|
||||
"eureka_defaults",
|
||||
],
|
||||
shared_libs: [
|
||||
"libbase",
|
||||
"libbinder_ndk",
|
||||
|
|
|
|||
|
|
@ -15,7 +15,6 @@ cc_binary {
|
|||
"service.cpp",
|
||||
],
|
||||
defaults: [
|
||||
"eureka_defaults",
|
||||
"fm_aidl_defaults",
|
||||
],
|
||||
static_libs: ["libfm_slsi-impl"],
|
||||
|
|
|
|||
|
|
@ -9,7 +9,6 @@ cc_library_static {
|
|||
shared_libs: ["libbase"],
|
||||
export_include_dirs: ["public"],
|
||||
defaults: [
|
||||
"eureka_defaults",
|
||||
"fm_aidl_defaults"
|
||||
],
|
||||
header_libs: ["logformat"],
|
||||
|
|
|
|||
|
|
@ -1,6 +1,5 @@
|
|||
cc_binary {
|
||||
name: "vendor.eureka.hardware.parts-service",
|
||||
defaults: ["eureka_defaults"],
|
||||
srcs: [
|
||||
"Battery.cpp",
|
||||
"FlashLight.cpp",
|
||||
|
|
|
|||
|
|
@ -19,6 +19,5 @@ cc_binary {
|
|||
srcs: [
|
||||
"Logger.cpp",
|
||||
],
|
||||
defaults: ["eureka_defaults"],
|
||||
init_rc: ["eklogger.rc"],
|
||||
}
|
||||
|
|
|
|||
|
|
@ -17,7 +17,6 @@ cc_binary {
|
|||
name: "android.hardware.camera.provider@2.5-service.exynos7885",
|
||||
defaults: [
|
||||
"hidl_defaults",
|
||||
"eureka_defaults",
|
||||
],
|
||||
compile_multilib: "32",
|
||||
proprietary: true,
|
||||
|
|
|
|||
|
|
@ -6,7 +6,6 @@ cc_binary {
|
|||
relative_install_path: "hw",
|
||||
vendor: true,
|
||||
defaults: [
|
||||
"eureka_defaults",
|
||||
"powerstats_pixel_defaults",
|
||||
],
|
||||
cflags: ["-Wno-thread-safety-negative"],
|
||||
|
|
|
|||
|
|
@ -15,7 +15,6 @@
|
|||
cc_library_static {
|
||||
name: "libinit_universal7885",
|
||||
recovery_available: true,
|
||||
defaults: ["eureka_defaults"],
|
||||
whole_static_libs: ["libbase"],
|
||||
header_libs: ["libbase_headers"],
|
||||
srcs: ["init_universal7885.cpp"],
|
||||
|
|
|
|||
Loading…
Reference in a new issue