commit 4df105635b626a894f16eaec92e763b773a5a6e1
author Chih-Hung Hsieh <chh@google.com> Wed Jul 27 16:05:50 2016 -0700
committer Chih-Hung Hsieh <chh@google.com> Wed Jul 27 16:05:50 2016 -0700
tree d362ffbf3cf20e02d364648a8074d11cd73a80c6
parent 02ccdc5db9bb39488a3fe22a907b3211c3a464b9

Fix clang-tidy performance warnings in healthd.

* Use const reference type for parameters to avoid unnecessary copy.

Bug: 30407689
Change-Id: I926ea7b6e9e020a228c812b7e23087a2b7ed9809
Test: build with WITH_TIDY=1

healthd/BatteryPropertiesRegistrar.cpp

diff --git a/healthd/BatteryPropertiesRegistrar.cpp b/healthd/BatteryPropertiesRegistrar.cpp
index 5d1fa52..d28ba41 100644
--- a/healthd/BatteryPropertiesRegistrar.cpp
+++ b/healthd/BatteryPropertiesRegistrar.cpp
@@ -35,7 +35,7 @@
     defaultServiceManager()->addService(String16("batteryproperties"), service);
 }
 
-void BatteryPropertiesRegistrar::notifyListeners(struct BatteryProperties props) {
+void BatteryPropertiesRegistrar::notifyListeners(const struct BatteryProperties& props) {
     Mutex::Autolock _l(mRegistrationLock);
     for (size_t i = 0; i < mListeners.size(); i++) {
         mListeners[i]->batteryPropertiesChanged(props);