diff options
author | Andreas Gampe <agampe@google.com> | 2014-11-13 14:31:11 -0800 |
---|---|---|
committer | Andreas Gampe <agampe@google.com> | 2014-11-13 14:31:11 -0800 |
commit | 347f2884dbb29efd9b996bbd7ae7ca7996eb5f08 (patch) | |
tree | 10d67ede51a9ec8dc83c981e4c1da5c5cfcddadd | |
parent | fecc0bf4f1d8d1939807123d0f01ca1fe377bda4 (diff) | |
download | frameworks_native-347f2884dbb29efd9b996bbd7ae7ca7996eb5f08.zip frameworks_native-347f2884dbb29efd9b996bbd7ae7ca7996eb5f08.tar.gz frameworks_native-347f2884dbb29efd9b996bbd7ae7ca7996eb5f08.tar.bz2 |
Frameworks/native: Wall Werror in batteryservice & powermanager
Turn on -Wall -Werror in services/batteryservice and services/powermanager.
Fix warnings.
Change-Id: I1da667083013379cd8240377c48d63fa5c53033d
-rw-r--r-- | services/batteryservice/Android.mk | 14 | ||||
-rw-r--r-- | services/batteryservice/IBatteryPropertiesListener.cpp | 2 | ||||
-rw-r--r-- | services/powermanager/Android.mk | 8 |
3 files changed, 14 insertions, 10 deletions
diff --git a/services/batteryservice/Android.mk b/services/batteryservice/Android.mk index 9354b99..e4097d7 100644 --- a/services/batteryservice/Android.mk +++ b/services/batteryservice/Android.mk @@ -2,17 +2,19 @@ LOCAL_PATH:= $(call my-dir) include $(CLEAR_VARS) LOCAL_SRC_FILES:= \ - BatteryProperties.cpp \ - BatteryProperty.cpp \ - IBatteryPropertiesListener.cpp \ - IBatteryPropertiesRegistrar.cpp + BatteryProperties.cpp \ + BatteryProperty.cpp \ + IBatteryPropertiesListener.cpp \ + IBatteryPropertiesRegistrar.cpp LOCAL_STATIC_LIBRARIES := \ - libutils \ - libbinder + libutils \ + libbinder LOCAL_MODULE:= libbatteryservice LOCAL_MODULE_TAGS := optional +LOCAL_CFLAGS += -Wall -Werror -Wunused -Wunreachable-code + include $(BUILD_STATIC_LIBRARY) diff --git a/services/batteryservice/IBatteryPropertiesListener.cpp b/services/batteryservice/IBatteryPropertiesListener.cpp index 19ac7f0..8aff26c 100644 --- a/services/batteryservice/IBatteryPropertiesListener.cpp +++ b/services/batteryservice/IBatteryPropertiesListener.cpp @@ -35,7 +35,7 @@ public: data.writeInterfaceToken(IBatteryPropertiesListener::getInterfaceDescriptor()); data.writeInt32(1); props.writeToParcel(&data); - status_t err = remote()->transact(TRANSACT_BATTERYPROPERTIESCHANGED, data, &reply, IBinder::FLAG_ONEWAY); + remote()->transact(TRANSACT_BATTERYPROPERTIESCHANGED, data, &reply, IBinder::FLAG_ONEWAY); } }; diff --git a/services/powermanager/Android.mk b/services/powermanager/Android.mk index d98b2da..7b24c65 100644 --- a/services/powermanager/Android.mk +++ b/services/powermanager/Android.mk @@ -2,14 +2,16 @@ LOCAL_PATH:= $(call my-dir) include $(CLEAR_VARS) LOCAL_SRC_FILES:= \ - IPowerManager.cpp + IPowerManager.cpp LOCAL_SHARED_LIBRARIES := \ - libutils \ - libbinder + libutils \ + libbinder LOCAL_MODULE:= libpowermanager LOCAL_MODULE_TAGS := optional +LOCAL_CFLAGS += -Wall -Werror -Wunused -Wunreachable-code + include $(BUILD_SHARED_LIBRARY) |