diff options
| author | Alexey Rusakov <Kitsune-Ral@users.sf.net> | 2021-01-25 21:49:52 +0100 |
|---|---|---|
| committer | Alexey Rusakov <Kitsune-Ral@users.sf.net> | 2021-01-25 21:49:52 +0100 |
| commit | cd71f81a964751cc820074bb345f904b22a2c583 (patch) | |
| tree | 2097d0b53e0829376ac4d9dd9b2c89a074de8ec7 /lib/room.cpp | |
| parent | ced197fc6606c0f12eee161408742da54f40411b (diff) | |
| download | libquotient-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/room.cpp')
| -rw-r--r-- | lib/room.cpp | 5 |
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); |