commit 65ab67312b70523095c4ed9b1a4596df50d8d0fd
author Chih-Hung Hsieh <chh@google.com> Wed Dec 12 14:13:48 2018 -0800
committer Chih-hung Hsieh <chh@google.com> Mon Dec 17 19:57:21 2018 +0000
tree 4a1b9d4b88590b26998f18f27da1cb050a4a9f48
parent f529ba8dac3b9e8ffd95bf21741074c7b91eac57

Fix performance-for-range-copy warnings

Bug: 30413223
Test: make with WITH_TIDY=1 DEFAULT_GLOBAL_TIDY_CHECKS=-*,performance*
Change-Id: I1b76a22eab100a31e77048626e48169fe7eeaf92

graphics/composer/2.1/utils/vts/ComposerVts.cpp

diff --git a/graphics/composer/2.1/utils/vts/ComposerVts.cpp b/graphics/composer/2.1/utils/vts/ComposerVts.cpp
index 2f531b4..6e668af 100644
--- a/graphics/composer/2.1/utils/vts/ComposerVts.cpp
+++ b/graphics/composer/2.1/utils/vts/ComposerVts.cpp
@@ -78,9 +78,9 @@
 ComposerClient::ComposerClient(const sp<IComposerClient>& client) : mClient(client) {}
 
 ComposerClient::~ComposerClient() {
-    for (auto it : mDisplayResources) {
+    for (const auto& it : mDisplayResources) {
         Display display = it.first;
-        DisplayResource& resource = it.second;
+        const DisplayResource& resource = it.second;
 
         for (auto layer : resource.layers) {
             EXPECT_EQ(Error::NONE, mClient->destroyLayer(display, layer))