From 17bf6cea9d896e0c2b2a08d0cbd666377afae1df Mon Sep 17 00:00:00 2001 From: Tristan Matthews <tristan.matthews@savoirfairelinux.com> Date: Tue, 13 Aug 2013 16:45:23 -0400 Subject: [PATCH] * #28351: history: use range-based for loops --- daemon/src/history/history.cpp | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/daemon/src/history/history.cpp b/daemon/src/history/history.cpp index 4b0d4e47ba..dab738de46 100644 --- a/daemon/src/history/history.cpp +++ b/daemon/src/history/history.cpp @@ -86,9 +86,8 @@ bool History::save() std::ofstream outfile(path_.c_str()); if (outfile.fail()) return false; - for (vector<HistoryItem>::const_iterator iter = items_.begin(); - iter != items_.end(); ++iter) - outfile << *iter << std::endl; + for (const auto &iter : items_) + outfile << iter << std::endl; return true; } @@ -129,9 +128,8 @@ vector<map<string, string> > History::getSerialized() { sfl::ScopedLock lock(historyItemsMutex_); vector<map<string, string> > result; - for (vector<HistoryItem>::const_iterator iter = items_.begin(); - iter != items_.end(); ++iter) - result.push_back(iter->toMap()); + for (const auto &iter : items_) + result.push_back(iter.toMap()); return result; } -- GitLab