diff options
author | Andrei Karas <akaras@inbox.ru> | 2013-09-18 23:07:40 +0300 |
---|---|---|
committer | Andrei Karas <akaras@inbox.ru> | 2013-09-18 23:07:40 +0300 |
commit | dc6568962a32ddb6f231c629e7ad3373e9086b6c (patch) | |
tree | 744e9f2507c6113e1ba941a11b16e4923aa00858 /src/being | |
parent | 6419386c4c382753509425826ee609dd07d4ae72 (diff) | |
download | mv-dc6568962a32ddb6f231c629e7ad3373e9086b6c.tar.gz mv-dc6568962a32ddb6f231c629e7ad3373e9086b6c.tar.bz2 mv-dc6568962a32ddb6f231c629e7ad3373e9086b6c.tar.xz mv-dc6568962a32ddb6f231c629e7ad3373e9086b6c.zip |
remove useless conditions.
Diffstat (limited to 'src/being')
-rw-r--r-- | src/being/being.cpp | 4 | ||||
-rw-r--r-- | src/being/localplayer.cpp | 12 |
2 files changed, 7 insertions, 9 deletions
diff --git a/src/being/being.cpp b/src/being/being.cpp index 3f40398a0..645a71dac 100644 --- a/src/being/being.cpp +++ b/src/being/being.cpp @@ -1062,7 +1062,7 @@ std::string Being::getWeaponAttackAction(const ItemInfo *const weapon) const if (!weapon) return SpriteAction::ATTACK; - if (serverVersion < 0 || !weapon) + if (serverVersion < 0) { return weapon->getAttackAction(); } @@ -1085,7 +1085,7 @@ std::string Being::getAttackAction(const Attack *const attack1) const if (!attack1) return SpriteAction::ATTACK; - if (serverVersion < 0 || !attack1) + if (serverVersion < 0) { return attack1->mAction; } diff --git a/src/being/localplayer.cpp b/src/being/localplayer.cpp index 04bb2603a..f78dd7329 100644 --- a/src/being/localplayer.cpp +++ b/src/being/localplayer.cpp @@ -990,7 +990,7 @@ void LocalPlayer::setDestination(const int x, const int y) Net::getPlayerHandler()->setDestination(x, y, mDirection); Being::setDestination(x, y); } - else if (mInvertDirection == 1) + else { uint8_t newDir = 0; if (mDirection & UP) @@ -1011,9 +1011,7 @@ void LocalPlayer::setDestination(const int x, const int y) } Being::setDestination(x, y); - } - else - { + #ifdef MANASERV_SUPPORT // Manaserv: // If the destination given to being class is accepted, @@ -1244,7 +1242,7 @@ void LocalPlayer::attack(Being *const target, const bool keep, if (!target || target->getType() == ActorSprite::NPC) return; - if (mTarget != target || !mTarget) + if (mTarget != target) setTarget(target); #ifdef MANASERV_SUPPORT @@ -3349,7 +3347,7 @@ void LocalPlayer::setHome() } else { - if (specialLayer && iter != mHomes.end()) + if (iter != mHomes.end()) { specialLayer->setTile(static_cast<int>(pos.x), static_cast<int>(pos.y), MapItem::EMPTY); @@ -3817,7 +3815,7 @@ void LocalPlayer::attack2(Being *const target, const bool keep, return; } setTarget(target); - if (target && target->getType() != Being::NPC) + if (target->getType() != Being::NPC) { mKeepAttacking = true; moveToTarget(); |