commit 41649d5bb219a3b316eae8af54673f54e0a75935
author Chih-Hung Hsieh <chh@google.com> Thu Aug 03 14:27:21 2017 -0700
committer Yifan Hong <elsk@google.com> Tue Aug 22 23:00:18 2017 +0000
tree 44e066a2bd07ca1c132d01269c672c55f1b0f255
parent 932de1ee3f10bf94d11b51cce2e92f3efaf6ef7c

Fix clang-tidy performance-* warnings in system/libhidl.

* Use const reference parameter type to avoid unnecessary copy.
* Use more efficient overloaded string methods.
* Use const reference type for loop index variables to avoid unnecessary copy.

Bug: 30407689
Bug: 30411878
Bug: 30413223
Test: build with WITH_TIDY=1
Test: libhidl_test
Change-Id: I1deae1f09d56c2fa9cd56728b13ca50305d18f3e

base/HidlInternal.cpp

diff --git a/base/HidlInternal.cpp b/base/HidlInternal.cpp
index 5fae1c4..0ff52ad 100644
--- a/base/HidlInternal.cpp
+++ b/base/HidlInternal.cpp
@@ -96,7 +96,7 @@
         instrumentationLibPaths.push_back(HAL_LIBRARY_PATH_ODM);
     }
 
-    for (auto path : instrumentationLibPaths) {
+    for (const auto& path : instrumentationLibPaths) {
         DIR *dir = opendir(path.c_str());
         if (dir == 0) {
             LOG(WARNING) << path << " does not exist. ";