[network/neochat/release/26.08] src/timeline/models: Rename parameters for clarity

Joshua Goins <[email protected]>
Newsgroups gmane.comp.kde.cvs
Message-ID <[email protected]>
Git commit 0cb508ae02109f1867103e458a49147fa801883a by Joshua Goins, on behalf of Tobias Fella.
Committed on 24/07/2026 at 20:14.
Pushed by redstrate into branch 'release/26.08'.

Rename parameters for clarity

(cherry picked from commit f93c6a81a0430783cb327d94c93067cf4cd6b2b5)

M  +5    -5    src/timeline/models/timelinemessagemodel.cpp

https://invent.kde.org/network/neochat/-/commit/0cb508ae02109f1867103e458a49147fa801883a

diff --git a/src/timeline/models/timelinemessagemodel.cpp b/src/timeline/models/timelinemessagemodel.cpp
index 56a852299..7768ed432 100644
--- a/src/timeline/models/timelinemessagemodel.cpp
+++ b/src/timeline/models/timelinemessagemodel.cpp
@@ -42,9 +42,9 @@ void TimelineMessageModel::connectNewRoom()
                 beginInsertRows({}, rowCount(), rowCount() + int(events.size()) - 1);
             }
         });
-        connect(m_room, &Room::addedMessages, this, [this](int lowest, int biggest) {
+        connect(m_room, &Room::addedMessages, this, [this](int oldest, int newest) {
             if (m_initialized) {
-                for (int i = lowest; i <= biggest; ++i) {
+                for (int i = oldest; i <= newest; ++i) {
                     const auto event = m_room->findInTimeline(i)->event();
                     Q_EMIT newEventAdded(event);
                 }
@@ -55,11 +55,11 @@ void TimelineMessageModel::connectNewRoom()
                 // no read marker, so see if we need to create one.
                 moveReadMarker(m_room->lastFullyReadEventId());
             }
-            if (biggest < m_room->maxTimelineIndex()) {
-                auto rowBelowInserted = m_room->maxTimelineIndex() - biggest + timelineServerIndex() - 1;
+            if (newest < m_room->maxTimelineIndex()) {
+                auto rowBelowInserted = m_room->maxTimelineIndex() - newest + timelineServerIndex() - 1;
                 refreshEventRoles(rowBelowInserted, {ContentModelRole});
             }
-            for (auto i = m_room->maxTimelineIndex() - biggest; i <= m_room->maxTimelineIndex() - lowest; ++i) {
+            for (auto i = m_room->maxTimelineIndex() - newest; i <= m_room->maxTimelineIndex() - oldest; ++i) {
                 refreshLastUserEvents(i);
             }
         });
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.