commit 0b2acdfcc99ecf3ff36fc6337e14b171170f38cf
author Elliott Hughes <enh@google.com> Fri Oct 02 18:25:19 2015 -0700
committer Elliott Hughes <enh@google.com> Fri Oct 02 18:25:19 2015 -0700
tree 1318497b782194c7ae533f88c1aabfca0affdc7d
parent f7cf8b2c1fbf6250e12d1e59aa353990eca4f109

Use const auto& in for loops.

Change-Id: Ic437c59797ee4e7dc38291da35c72d827bc89c8d

tests/pthread_test.cpp

diff --git a/tests/pthread_test.cpp b/tests/pthread_test.cpp
index 1766762..9f887e3 100644
--- a/tests/pthread_test.cpp
+++ b/tests/pthread_test.cpp
@@ -70,7 +70,7 @@
   std::vector<pthread_key_t> keys;
 
   auto scope_guard = make_scope_guard([&keys]{
-    for (auto key : keys) {
+    for (const auto& key : keys) {
       EXPECT_EQ(0, pthread_key_delete(key));
     }
   });
@@ -108,7 +108,7 @@
   }
 
   // Don't leak keys.
-  for (auto key : keys) {
+  for (const auto& key : keys) {
     EXPECT_EQ(0, pthread_key_delete(key));
   }
   keys.clear();
@@ -1162,7 +1162,7 @@
   void* maps_stack_hi = NULL;
   std::vector<map_record> maps;
   ASSERT_TRUE(Maps::parse_maps(&maps));
-  for (auto& map : maps) {
+  for (const auto& map : maps) {
     if (map.pathname == "[stack]") {
       maps_stack_hi = reinterpret_cast<void*>(map.addr_end);
       break;
@@ -1552,8 +1552,8 @@
   }
 
   ~StrictAlignmentAllocator() {
-    for (auto& p : allocated_array) {
-      delete [] p;
+    for (const auto& p : allocated_array) {
+      delete[] p;
     }
   }