aboutsummaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorAlexey Rusakov <Kitsune-Ral@users.sf.net>2021-01-25 21:49:52 +0100
committerAlexey Rusakov <Kitsune-Ral@users.sf.net>2021-01-25 21:49:52 +0100
commitcd71f81a964751cc820074bb345f904b22a2c583 (patch)
tree2097d0b53e0829376ac4d9dd9b2c89a074de8ec7 /lib
parentced197fc6606c0f12eee161408742da54f40411b (diff)
downloadlibquotient-cd71f81a964751cc820074bb345f904b22a2c583.tar.gz
libquotient-cd71f81a964751cc820074bb345f904b22a2c583.zip
Room::P::removeMemberFromMap: comment that Q_ASSERT
Maybe it's not even that bad, given that an effort is taken to recover from the internal member list corruption.
Diffstat (limited to 'lib')
-rw-r--r--lib/room.cpp5
1 files changed, 3 insertions, 2 deletions
diff --git a/lib/room.cpp b/lib/room.cpp
index 2e8641aa..fadcea17 100644
--- a/lib/room.cpp
+++ b/lib/room.cpp
@@ -1404,8 +1404,9 @@ void Room::Private::removeMemberFromMap(User* u)
if (et.nsecsElapsed() > profilerMinNsecs() / 10)
qCDebug(MEMBERS) << "...done in" << et;
if (it != membersMap.cend()) {
- Q_ASSERT_X(false, __FUNCTION__,
- "Mismatched name in the room members list");
+ // The assert (still) does more harm than good, it seems
+// Q_ASSERT_X(false, __FUNCTION__,
+// "Mismatched name in the room members list");
qCCritical(MEMBERS) << "Mismatched name in the room members list;"
" avoiding the list corruption";
membersMap.remove(it.key(), u);