diff options
author | Thorbjørn Lindeijer <bjorn@lindeijer.nl> | 2024-01-29 21:53:25 +0100 |
---|---|---|
committer | Thorbjørn Lindeijer <bjorn@lindeijer.nl> | 2024-01-30 12:32:55 +0100 |
commit | 2d97a4b9d7c6d84cb8b22302cd22f805a4ab76b0 (patch) | |
tree | 77453315596574d874821f78075ffabb6731e76c /src/gui/setup_players.cpp | |
parent | 87470d984cc8f6fe12a354aec7bf93ee4fcb95eb (diff) | |
download | mana-2d97a4b9d7c6d84cb8b22302cd22f805a4ab76b0.tar.gz mana-2d97a4b9d7c6d84cb8b22302cd22f805a4ab76b0.tar.bz2 mana-2d97a4b9d7c6d84cb8b22302cd22f805a4ab76b0.tar.xz mana-2d97a4b9d7c6d84cb8b22302cd22f805a4ab76b0.zip |
Apply readability-delete-null-pointer fixits
These checks are unnecessary as deleting a null pointer has no effect.
Diffstat (limited to 'src/gui/setup_players.cpp')
-rw-r--r-- | src/gui/setup_players.cpp | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/src/gui/setup_players.cpp b/src/gui/setup_players.cpp index fe72a29c..529e69a5 100644 --- a/src/gui/setup_players.cpp +++ b/src/gui/setup_players.cpp @@ -127,8 +127,8 @@ public: signalBeforeUpdate(); freeWidgets(); - if (mPlayers) - delete mPlayers; + + delete mPlayers; mPlayers = player_relations.getPlayers(); // set up widgets @@ -162,8 +162,7 @@ public: virtual void freeWidgets() { - if (mPlayers) - delete mPlayers; + delete mPlayers; mPlayers = nullptr; delete_all(mWidgets); |