[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);
}
});