summaryrefslogtreecommitdiff
path: root/src/being
diff options
context:
space:
mode:
authorAndrei Karas <akaras@inbox.ru>2013-09-18 23:07:40 +0300
committerAndrei Karas <akaras@inbox.ru>2013-09-18 23:07:40 +0300
commitdc6568962a32ddb6f231c629e7ad3373e9086b6c (patch)
tree744e9f2507c6113e1ba941a11b16e4923aa00858 /src/being
parent6419386c4c382753509425826ee609dd07d4ae72 (diff)
downloadmv-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.cpp4
-rw-r--r--src/being/localplayer.cpp12
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();