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/being/actor.cpp | |
parent | f1518dd8476c968a43fa57cfb06198e290a4f77a (diff) | |
download | manaverse-36ba43d6ea38062b17f7e63ef659962bfc51c64d.tar.gz manaverse-36ba43d6ea38062b17f7e63ef659962bfc51c64d.tar.bz2 manaverse-36ba43d6ea38062b17f7e63ef659962bfc51c64d.tar.xz manaverse-36ba43d6ea38062b17f7e63ef659962bfc51c64d.zip |
Fix clang-tidy check readability-implicit-bool-cast.
Diffstat (limited to 'src/being/actor.cpp')
-rw-r--r-- | src/being/actor.cpp | 16 |
1 files changed, 11 insertions, 5 deletions
diff --git a/src/being/actor.cpp b/src/being/actor.cpp index 9039c0128..a881c005e 100644 --- a/src/being/actor.cpp +++ b/src/being/actor.cpp @@ -37,7 +37,7 @@ Actor::Actor() : Actor::~Actor() { - if (mMap) + if (mMap != nullptr) { mMap->removeActor(mMapActor); mMap = nullptr; @@ -47,28 +47,34 @@ Actor::~Actor() void Actor::setMap(Map *const map) { // Remove Actor from potential previous map - if (mMap) + if (mMap != nullptr) mMap->removeActor(mMapActor); mMap = map; // Add Actor to potential new map - if (mMap) + if (mMap != nullptr) mMapActor = mMap->addActor(this); } int Actor::getTileX() const { - if (!mMap || !mMap->getTileWidth()) + if (mMap == nullptr || + mMap->getTileWidth() == 0) + { return 0; + } return getPixelX() / mMap->getTileWidth(); } int Actor::getTileY() const { - if (!mMap || !mMap->getTileHeight()) + if (mMap == nullptr || + mMap->getTileHeight() == 0) + { return 0; + } return getPixelY() / mMap->getTileHeight(); } |