More code cleanup and tweaks; fine-tuning logs; performance improvements

After adding some profiling it became clear that to recalculate the room name and emit namesChanged() upon each member event is a waste, especially when there are thousands of those coming at initial sync (*cough* Matrix HQ room). So the room name is recalculated only once and unconditionally (in most cases this will boil down to checking whether name/canonicalAlias changed after processing the events batch), and namesChanged is only emitted once per batch, if any name or alias changed.

1 files changed, 2 insertions, 2 deletions

diff --git a/room.h b/room.h
index 689a99b0..60be33b9 100644
--- a/room.h
+++ b/room.h
@@ -102,7 +102,7 @@ namespace QMatrixClient
              */
             Q_INVOKABLE QString roomMembername(const QString& userId) const;
 
-            Q_INVOKABLE void updateData(SyncRoomData& data );
+            void updateData(SyncRoomData&& data );
             Q_INVOKABLE void setJoinState( JoinState state );
 
             const Timeline& messageEvents() const;
@@ -116,7 +116,7 @@ namespace QMatrixClient
             Q_INVOKABLE bool isValidIndex(TimelineItem::index_t timelineIndex) const;
 
             rev_iter_t findInTimeline(TimelineItem::index_t index) const;
-            rev_iter_t findInTimeline(QString evtId) const;
+            rev_iter_t findInTimeline(const QString& evtId) const;
 
             rev_iter_t readMarker(const User* user) const;
             rev_iter_t readMarker() const;