commit 020600db42428e7e96bb0d2e05da225e68650955
author Alex Deymo <deymo@chromium.org> Wed Nov 05 21:05:55 2014 -0800
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Fri Nov 07 19:23:03 2014 +0000
tree c5e1440655b9bc05570a20721cdf8171502862b7
parent 98c5ae2c999c14e63da06a3e6a389a9f9e9f291f

update_engine: Use C++11 range-based for loop when possible.

This cleanup just simplifies the notation of several for loops using
the new range-based syntax.

BUG=None
TEST=`FEATURES=test USE="lang clang" emerge-link update_engine`

Change-Id: I2120a1b1c3fd00e9a02a62abfa8427932080d0f9
Reviewed-on: https://chromium-review.googlesource.com/227897
Tested-by: Alex Deymo <deymo@chromium.org>
Reviewed-by: Alex Vakulenko <avakulenko@chromium.org>
Commit-Queue: Alex Deymo <deymo@chromium.org>

update_attempter.cc

diff --git a/update_attempter.cc b/update_attempter.cc
index 200493d..32c3728 100644
--- a/update_attempter.cc
+++ b/update_attempter.cc
@@ -673,9 +673,8 @@
   actions_.push_back(shared_ptr<AbstractAction>(update_complete_action));
 
   // Enqueue the actions
-  for (vector<shared_ptr<AbstractAction>>::iterator it = actions_.begin();
-       it != actions_.end(); ++it) {
-    processor_->EnqueueAction(it->get());
+  for (const shared_ptr<AbstractAction>& action : actions_) {
+    processor_->EnqueueAction(action.get());
   }
 }
 
@@ -727,9 +726,8 @@
   BuildPostInstallActions(install_plan_action.get());
 
   // Enqueue the actions
-  for (vector<shared_ptr<AbstractAction>>::iterator it = actions_.begin();
-       it != actions_.end(); ++it) {
-    processor_->EnqueueAction(it->get());
+  for (const shared_ptr<AbstractAction>& action : actions_) {
+    processor_->EnqueueAction(action.get());
   }
 
   // Update the payload state for Rollback.