From e6b45263bcc25a7e5e5cb9d880a735507b88a48e Mon Sep 17 00:00:00 2001
From: Konstantinos Sideris <sideris.konstantin@gmail.com>
Date: Sun, 15 Jul 2018 18:58:09 +0300
Subject: [PATCH] User reverse iterator instead of while loop

---
 src/timeline/TimelineView.cc | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/src/timeline/TimelineView.cc b/src/timeline/TimelineView.cc
index ca2f6211..967436ff 100644
--- a/src/timeline/TimelineView.cc
+++ b/src/timeline/TimelineView.cc
@@ -380,13 +380,9 @@ TimelineView::renderTopEvents(const std::vector<TimelineEvent> &events)
         firstSender_.clear();
         firstMsgTimestamp_ = QDateTime();
 
-        // Parse in reverse order to determine where we should not show sender's
-        // name.
-        auto ii = events.size();
-        while (ii != 0) {
-                --ii;
-
-                auto item = parseMessageEvent(events[ii], TimelineDirection::Top);
+        // Parse in reverse order to determine where we should not show sender's name.
+        for (auto it = events.rbegin(); it != events.rend(); ++it) {
+                auto item = parseMessageEvent(*it, TimelineDirection::Top);
 
                 if (item != nullptr)
                         items.push_back(item);