diff options
| author | Andrei Karas <akaras@inbox.ru> | 2017-06-06 23:34:34 +0300 |
|---|---|---|
| committer | Andrei Karas <akaras@inbox.ru> | 2017-06-07 19:23:40 +0300 |
| commit | 36ba43d6ea38062b17f7e63ef659962bfc51c64d (patch) | |
| tree | 190156cb88b13a38a6d13c69ee0742cc078065a1 /src/gui/models/playertablemodel.cpp | |
| parent | f1518dd8476c968a43fa57cfb06198e290a4f77a (diff) | |
| download | plus-36ba43d6ea38062b17f7e63ef659962bfc51c64d.tar.gz plus-36ba43d6ea38062b17f7e63ef659962bfc51c64d.tar.bz2 plus-36ba43d6ea38062b17f7e63ef659962bfc51c64d.tar.xz plus-36ba43d6ea38062b17f7e63ef659962bfc51c64d.zip | |
Fix clang-tidy check readability-implicit-bool-cast.
Diffstat (limited to 'src/gui/models/playertablemodel.cpp')
| -rw-r--r-- | src/gui/models/playertablemodel.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/gui/models/playertablemodel.cpp b/src/gui/models/playertablemodel.cpp index 29e08578d..9c8526e0d 100644 --- a/src/gui/models/playertablemodel.cpp +++ b/src/gui/models/playertablemodel.cpp @@ -66,7 +66,7 @@ PlayerTableModel::~PlayerTableModel() int PlayerTableModel::getRows() const { - if (mPlayers) + if (mPlayers != nullptr) return CAST_S32(mPlayers->size()); else return 0; @@ -120,7 +120,7 @@ void PlayerTableModel::updateModelInRow(const int row) const { const DropDown *const choicebox = static_cast<DropDown *>( getElementAt(row, RELATION_CHOICE_COLUMN)); - if (!choicebox) + if (choicebox == nullptr) return; player_relations.setRelation(getPlayerAt(row), static_cast<RelationT>( @@ -141,7 +141,7 @@ void PlayerTableModel::freeWidgets() std::string PlayerTableModel::getPlayerAt(const int index) const { - if (!mPlayers || index < 0 + if ((mPlayers == nullptr) || index < 0 || index >= CAST_S32(mPlayers->size())) { return std::string(); |