Use -Werror in hardware/libhardware_legacy
* Remove unused local variable or suppress the warning. Bug: 66996870 Test: build with WITH_TIDY=1 Change-Id: I4d7b4f2e9e7e6c60d8bfc209f47e6f1d33720165
This commit is contained in:
parent
53401d7c35
commit
3ba7298e95
3 changed files with 6 additions and 1 deletions
|
@ -17,6 +17,7 @@ cc_library {
|
||||||
name: "libpower",
|
name: "libpower",
|
||||||
|
|
||||||
srcs: ["power.c"],
|
srcs: ["power.c"],
|
||||||
|
cflags: ["-Wall", "-Werror"],
|
||||||
export_include_dirs: ["include"],
|
export_include_dirs: ["include"],
|
||||||
shared_libs: ["libcutils", "liblog"],
|
shared_libs: ["libcutils", "liblog"],
|
||||||
vendor_available: true,
|
vendor_available: true,
|
||||||
|
@ -48,6 +49,8 @@ cc_library_shared {
|
||||||
|
|
||||||
cflags: [
|
cflags: [
|
||||||
"-DQEMU_HARDWARE",
|
"-DQEMU_HARDWARE",
|
||||||
|
"-Wall",
|
||||||
|
"-Werror",
|
||||||
"-Wno-unused-parameter",
|
"-Wno-unused-parameter",
|
||||||
"-Wno-gnu-designator",
|
"-Wno-gnu-designator",
|
||||||
],
|
],
|
||||||
|
|
|
@ -13,7 +13,10 @@ cc_library_static {
|
||||||
name: "libaudiohw_legacy",
|
name: "libaudiohw_legacy",
|
||||||
static_libs: ["libmedia_helper"],
|
static_libs: ["libmedia_helper"],
|
||||||
cflags: [
|
cflags: [
|
||||||
|
"-Wall",
|
||||||
|
"-Werror",
|
||||||
"-Wno-unused-parameter",
|
"-Wno-unused-parameter",
|
||||||
|
"-Wno-unused-variable",
|
||||||
"-Wno-gnu-designator",
|
"-Wno-gnu-designator",
|
||||||
],
|
],
|
||||||
|
|
||||||
|
|
1
power.c
1
power.c
|
@ -91,7 +91,6 @@ acquire_wake_lock(int lock, const char* id)
|
||||||
if (g_error) return g_error;
|
if (g_error) return g_error;
|
||||||
|
|
||||||
int fd;
|
int fd;
|
||||||
size_t len;
|
|
||||||
ssize_t ret;
|
ssize_t ret;
|
||||||
|
|
||||||
if (lock != PARTIAL_WAKE_LOCK) {
|
if (lock != PARTIAL_WAKE_LOCK) {
|
||||||
|
|
Loading…
Reference in a new issue