diff options
| author | Kitsune Ral <Kitsune-Ral@users.sf.net> | 2018-12-16 14:10:05 +0900 |
|---|---|---|
| committer | Kitsune Ral <Kitsune-Ral@users.sf.net> | 2018-12-16 14:22:17 +0900 |
| commit | bf5401753432533b31e7d18519c2031c84e774b7 (patch) | |
| tree | 404ff94211954289935be27a8b46ff911182efcc /lib | |
| parent | fcc8d2ca615fce6e42bf1cf6585d60f94a2db926 (diff) | |
| download | libquotient-bf5401753432533b31e7d18519c2031c84e774b7.tar.gz libquotient-bf5401753432533b31e7d18519c2031c84e774b7.zip | |
Room::getAllMembers: revert off-by-one "bugfix"
It actually introduces an off-by-one error; the original code was
correct. #qmatrixclient:matrix.org is used instead of #test:matrix.org
to check lazy-loading (see https://github.com/matrix-org/synapse/issues/4300)
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/room.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/room.cpp b/lib/room.cpp index 8f9095dd..7232741a 100644 --- a/lib/room.cpp +++ b/lib/room.cpp @@ -595,7 +595,7 @@ Room::rev_iter_t Room::findInTimeline(const QString& evtId) const void Room::Private::getAllMembers() { // If already loaded or already loading, there's nothing to do here. - if (q->joinedCount() - 1 <= membersMap.size() || isJobRunning(allMembersJob)) + if (q->joinedCount() <= membersMap.size() || isJobRunning(allMembersJob)) return; allMembersJob = connection->callApi<GetMembersByRoomJob>( |