commit 1f601b1d7d5f94c6f8f7850e14d1b88e7fb9a775
author Chih-Hung Hsieh <chh@google.com> Tue Nov 14 16:05:46 2017 -0800
committer Chih-Hung Hsieh <chh@google.com> Fri Dec 01 10:37:19 2017 -0800
tree c5b727ec0b0476d83bc8c35bfc44372803163b47
parent 49cf86194541314ec6e42a0ef607d04cd39c36c4

Use -Werror in hardware/libhardware/modules

* Remove unused local variables.
* Remove or comment out unused static functions.
* Fix trivial bugs found by unused value warnings.

Bug: 66996870
Test: build with WITH_TIDY=1
Change-Id: I99389b883c89551850180d25241a35a40bb77b26

modules/sensors/Android.mk

diff --git a/modules/sensors/Android.mk b/modules/sensors/Android.mk
index ca277e6..5ef6a97 100644
--- a/modules/sensors/Android.mk
+++ b/modules/sensors/Android.mk
@@ -25,7 +25,7 @@
 LOCAL_MODULE_RELATIVE_PATH := hw
 LOCAL_PROPRIETARY_MODULE := true
 
-LOCAL_CFLAGS := -DLOG_TAG=\"MultiHal\"
+LOCAL_CFLAGS := -Wall -Werror -DLOG_TAG=\"MultiHal\"
 
 LOCAL_SRC_FILES := \
     multihal.cpp \

modules/sensors/tests/Android.mk

diff --git a/modules/sensors/tests/Android.mk b/modules/sensors/tests/Android.mk
index 010bb90..dcf4f24 100644
--- a/modules/sensors/tests/Android.mk
+++ b/modules/sensors/tests/Android.mk
@@ -5,7 +5,7 @@
 LOCAL_SRC_FILES := \
 	SensorEventQueue_test.cpp
 
-#LOCAL_CFLAGS := -g
+LOCAL_CFLAGS := -Wall -Werror
 LOCAL_MODULE := sensorstests
 
 LOCAL_STATIC_LIBRARIES := libcutils libutils