diff options
| author | jackpoz <giacomopoz@gmail.com> | 2018-02-17 23:26:50 +0100 |
|---|---|---|
| committer | funjoker <funjoker109@gmail.com> | 2021-08-08 21:21:34 +0200 |
| commit | b7aef385e863072eef544dcfcc033090a882ff8f (patch) | |
| tree | 1b3b4002598d6dffd2c170492e2adf72be488e44 /src/server/game/Groups/Group.cpp | |
| parent | cb0a889177d36c6d054165e8d26774dbf9dd21d7 (diff) | |
Core/Misc: ASSERT() cleanup
Move some functions calls out of ASSERT() calls. ASSERT() should only apply checks without modifying any object and without having any side effect.
(cherry picked from commit 572eab1c355d9298ee640ebd7439f55366224fb8)
Diffstat (limited to 'src/server/game/Groups/Group.cpp')
| -rw-r--r-- | src/server/game/Groups/Group.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/server/game/Groups/Group.cpp b/src/server/game/Groups/Group.cpp index a9fd5085d11..07be71b46ff 100644 --- a/src/server/game/Groups/Group.cpp +++ b/src/server/game/Groups/Group.cpp @@ -169,7 +169,8 @@ bool Group::Create(Player* leader) Group::ConvertLeaderInstancesToGroup(leader, this, false); - ASSERT(AddMember(leader)); // If the leader can't be added to a new group because it appears full, something is clearly wrong. + bool addMemberResult = AddMember(leader); + ASSERT(addMemberResult); // If the leader can't be added to a new group because it appears full, something is clearly wrong. } else if (!AddMember(leader)) return false; |