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.h

diff --git a/healthd/BatteryPropertiesRegistrar.h b/healthd/BatteryPropertiesRegistrar.h
index d17e4a3..095f3d3 100644
--- a/healthd/BatteryPropertiesRegistrar.h
+++ b/healthd/BatteryPropertiesRegistrar.h
@@ -31,7 +31,7 @@
                                    public IBinder::DeathRecipient {
 public:
     void publish(const sp<BatteryPropertiesRegistrar>& service);
-    void notifyListeners(struct BatteryProperties props);
+    void notifyListeners(const struct BatteryProperties& props);
 
 private:
     Mutex mRegistrationLock;