Remove dead code & small refactorings using the std library

2 files changed, 11 insertions, 19 deletions

diff --git a/src/timeline/TimelineView.cc b/src/timeline/TimelineView.cc

index 999d608c..33da7f82 100644
--- a/src/timeline/TimelineView.cc
+++ b/src/timeline/TimelineView.cc
@@ -21,7 +21,6 @@
 
 #include "Config.h"
 #include "FloatingButton.h"
-#include "RoomMessages.h"
 #include "Utils.h"
 
 #include "timeline/TimelineView.h"
@@ -578,17 +577,12 @@ TimelineView::isPendingMessage(const QString &txnid,
         if (sender != local_userid)
                 return false;
 
-        for (const auto &msg : pending_msgs_) {
-                if (QString::number(msg.txn_id) == txnid)
-                        return true;
-        }
-
-        for (const auto &msg : pending_sent_msgs_) {
-                if (QString::number(msg.txn_id) == txnid)
-                        return true;
-        }
+        auto match_txnid = [txnid](const auto &msg) -> bool {
+                return QString::number(msg.txn_id) == txnid;
+        };
 
-        return false;
+        return std::any_of(pending_msgs_.cbegin(), pending_msgs_.cend(), match_txnid) ||
+               std::any_of(pending_sent_msgs_.cbegin(), pending_sent_msgs_.cend(), match_txnid);
 }
 
 void
diff --git a/src/timeline/TimelineViewManager.cc b/src/timeline/TimelineViewManager.cc

index 3fee59c7..ccbf509b 100644
--- a/src/timeline/TimelineViewManager.cc
+++ b/src/timeline/TimelineViewManager.cc
@@ -206,8 +206,8 @@ TimelineViewManager::addRoom(const QString &room_id)
 void
 TimelineViewManager::sync(const mtx::responses::Rooms &rooms)
 {
-        for (auto it = rooms.join.cbegin(); it != rooms.join.cend(); ++it) {
-                auto roomid = QString::fromStdString(it->first);
+        for (const auto &room : rooms.join) {
+                auto roomid = QString::fromStdString(room.first);
 
                 if (!timelineViewExists(roomid)) {
                         qDebug() << "Ignoring event from unknown room" << roomid;
@@ -216,7 +216,7 @@ TimelineViewManager::sync(const mtx::responses::Rooms &rooms)
 
                 auto view = views_.at(roomid);
 
-                view->addEvents(it->second.timeline);
+                view->addEvents(room.second.timeline);
         }
 }
 
@@ -309,9 +309,7 @@ TimelineViewManager::displayName(const QString &userid)
 bool
 TimelineViewManager::hasLoaded() const
 {
-        for (const auto &view : views_)
-                if (!view.second->hasLoaded())
-                        return false;
-
-        return true;
+        return std::all_of(views_.cbegin(), views_.cend(), [](const auto &view) {
+                return view.second->hasLoaded();
+        });
 }