summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndrei Karas <akaras@inbox.ru>2016-03-05 19:17:59 +0300
committerAndrei Karas <akaras@inbox.ru>2016-03-05 19:17:59 +0300
commit1a9f3cc7de6e7e304290d2979eefbf9b0239a968 (patch)
treea583bdcac59f4dcf882d435c58928344c0b9c352
parentb19a1c6d50c2c0f4233972f20c9354efca5ccb66 (diff)
downloadplus-1a9f3cc7de6e7e304290d2979eefbf9b0239a968.tar.gz
plus-1a9f3cc7de6e7e304290d2979eefbf9b0239a968.tar.bz2
plus-1a9f3cc7de6e7e304290d2979eefbf9b0239a968.tar.xz
plus-1a9f3cc7de6e7e304290d2979eefbf9b0239a968.zip
Replace most getters callc getTileX/getTileY to direct variables usage.
-rw-r--r--src/being/being.cpp20
-rw-r--r--src/being/being.h2
-rw-r--r--src/being/localplayer.cpp66
-rw-r--r--src/being/localplayer.h4
4 files changed, 46 insertions, 46 deletions
diff --git a/src/being/being.cpp b/src/being/being.cpp
index 2e1302fb7..127c38232 100644
--- a/src/being/being.cpp
+++ b/src/being/being.cpp
@@ -942,8 +942,8 @@ void Being::handleAttack(Being *restrict const victim,
if (this != localPlayer)
setAction(BeingAction::ATTACK, attackId);
- mLastAttackX = victim->getTileX();
- mLastAttackY = victim->getTileY();
+ mLastAttackX = victim->mX;
+ mLastAttackY = victim->mY;
if (mType == ActorType::Player && mEquippedWeapon)
fireMissile(victim, mEquippedWeapon->getMissileParticleFile());
@@ -957,8 +957,8 @@ void Being::handleAttack(Being *restrict const victim,
!serverFeatures->haveAttackDirections() &&
this != localPlayer)
{
- const uint8_t dir = calcDirection(victim->getTileX(),
- victim->getTileY());
+ const uint8_t dir = calcDirection(victim->mX,
+ victim->mY);
if (dir)
setDirection(dir);
}
@@ -1021,8 +1021,8 @@ void Being::handleSkill(Being *restrict const victim,
if (!serverFeatures->haveAttackDirections() && this != localPlayer)
{
- const uint8_t dir = calcDirection(victim->getTileX(),
- victim->getTileY());
+ const uint8_t dir = calcDirection(victim->mX,
+ victim->mY);
if (dir)
setDirection(dir);
}
@@ -1896,8 +1896,8 @@ void Being::petLogic() restrict2
mMoveTime = time;
- const int dstX0 = mOwner->getTileX();
- const int dstY0 = mOwner->getTileY();
+ const int dstX0 = mOwner->mX;
+ const int dstY0 = mOwner->mY;
int dstX = dstX0;
int dstY = dstY0;
const int followDist = mInfo->getStartFollowDist();
@@ -3900,8 +3900,8 @@ void Being::fixPetSpawnPos(int &restrict dstX,
{
if (!mMap->getWalk(dstX, dstY, getBlockWalkMask()))
{
- dstX = mOwner->getTileX();
- dstY = mOwner->getTileY();
+ dstX = mOwner->mX;
+ dstY = mOwner->mY;
}
}
}
diff --git a/src/being/being.h b/src/being/being.h
index ce1b27ebd..efac91bd4 100644
--- a/src/being/being.h
+++ b/src/being/being.h
@@ -1062,7 +1062,7 @@ class Being notfinal : public ActorSprite,
bool mShowName;
bool mIsGM;
- private:
+ protected:
/**
* Calculates the offset in the given directions.
* If walking in direction 'neg' the value is negated.
diff --git a/src/being/localplayer.cpp b/src/being/localplayer.cpp
index 47735875c..adda425b4 100644
--- a/src/being/localplayer.cpp
+++ b/src/being/localplayer.cpp
@@ -302,9 +302,9 @@ void LocalPlayer::logic()
{
// Find whether target is in range
const int rangeX = CAST_S32(
- abs(mTarget->getTileX() - getTileX()));
+ abs(mTarget->mX - mX));
const int rangeY = CAST_S32(
- abs(mTarget->getTileY() - getTileY()));
+ abs(mTarget->mY - mY));
const int attackRange = getAttackRange();
const TargetCursorTypeT targetType
= rangeX > attackRange || rangeY > attackRange
@@ -551,8 +551,8 @@ void LocalPlayer::setTarget(Being *const target)
if (mTarget)
{
- mLastTargetX = mTarget->getTileX();
- mLastTargetY = mTarget->getTileY();
+ mLastTargetX = mTarget->mX;
+ mLastTargetY = mTarget->mY;
mTarget->updateName();
}
@@ -763,8 +763,8 @@ void LocalPlayer::attack(Being *const target, const bool keep,
if (mTarget != target)
setTarget(target);
- const int dist_x = target->getTileX() - mX;
- const int dist_y = target->getTileY() - mY;
+ const int dist_x = target->mX - mX;
+ const int dist_y = target->mY - mY;
// Must be standing or sitting or casting to attack
if (mAction != BeingAction::STAND &&
@@ -975,8 +975,8 @@ bool LocalPlayer::withinAttackRange(const Being *const target,
if (fixDistance && range == 1)
range = 2;
- dx = CAST_S32(abs(target->getTileX() - mX));
- dy = CAST_S32(abs(target->getTileY() - mY));
+ dx = CAST_S32(abs(target->mX - mX));
+ dy = CAST_S32(abs(target->mY - mY));
return !(dx > range || dy > range);
}
@@ -988,7 +988,8 @@ void LocalPlayer::setGotoTarget(Being *const target)
mPickUpTarget = nullptr;
setTarget(target);
mGoingToTarget = true;
- setDestination(target->getTileX(), target->getTileY());
+ setDestination(target->mX,
+ target->mY);
}
void LocalPlayer::handleStatusEffect(const StatusEffect *const effect,
@@ -1240,8 +1241,8 @@ void LocalPlayer::moveToTarget(int dist)
debugPath = mMap->findPath(
(mPixelX - mapTileSize / 2) / mapTileSize,
(mPixelY - mapTileSize) / mapTileSize,
- mTarget->getTileX(),
- mTarget->getTileY(),
+ mTarget->mX,
+ mTarget->mY,
getBlockWalkMask(),
0);
}
@@ -1264,7 +1265,7 @@ void LocalPlayer::moveToTarget(int dist)
if (dist == 0)
{
if (mTarget)
- navigateTo(mTarget->getTileX(), mTarget->getTileY());
+ navigateTo(mTarget->mX, mTarget->mY);
}
else
{
@@ -1326,8 +1327,8 @@ void LocalPlayer::changeEquipmentBeforeAttack(const Being *const target) const
}
bool allowSword = false;
- const int dx = target->getTileX() - mX;
- const int dy = target->getTileY() - mY;
+ const int dx = target->mX - mX;
+ const int dy = target->mY - mY;
const Item *item = nullptr;
if (dx * dx + dy * dy > 80)
@@ -1405,17 +1406,17 @@ bool LocalPlayer::isReachable(Being *const being,
if (being->getReachable() == Reachable::REACH_NO)
return false;
- if (being->getTileX() == mX
- && being->getTileY() == mY)
+ if (being->mX == mX &&
+ being->mY == mY)
{
being->setDistance(0);
being->setReachable(Reachable::REACH_YES);
return true;
}
- else if (being->getTileX() - 1 <= mX
- && being->getTileX() + 1 >= mX
- && being->getTileY() - 1 <= mY
- && being->getTileY() + 1 >= mY)
+ else if (being->mX - 1 <= mX &&
+ being->mX + 1 >= mX &&
+ being->mY - 1 <= mY &&
+ being->mY + 1 >= mY)
{
being->setDistance(1);
being->setReachable(Reachable::REACH_YES);
@@ -1425,8 +1426,8 @@ bool LocalPlayer::isReachable(Being *const being,
const Path debugPath = mMap->findPath(
(mPixelX - mapTileSize / 2) / mapTileSize,
(mPixelY - mapTileSize) / mapTileSize,
- being->getTileX(),
- being->getTileY(),
+ being->mX,
+ being->mY,
getBlockWalkMask(),
maxCost);
@@ -2070,8 +2071,8 @@ void LocalPlayer::updateCoords()
navigateClean();
return;
}
- mNavigateX = being->getTileX();
- mNavigateY = being->getTileY();
+ mNavigateX = being->mX;
+ mNavigateY = being->mY;
}
if (mNavigateX == x && mNavigateY == y)
@@ -2145,8 +2146,8 @@ int LocalPlayer::getPathLength(const Being *const being) const
const Path debugPath = mMap->findPath(
(mPixelX - mapTileSize / 2) / mapTileSize,
(mPixelY - mapTileSize) / mapTileSize,
- being->getTileX(),
- being->getTileY(),
+ being->mX,
+ being->mY,
getBlockWalkMask(),
0);
return CAST_S32(debugPath.size());
@@ -2488,10 +2489,9 @@ void LocalPlayer::setRealPos(const int x, const int y)
SpecialLayer *const layer = mMap->getTempLayer();
if (layer)
{
- if ((mCrossX || mCrossY)
- && layer->getTile(mCrossX, mCrossY)
- && layer->getTile(mCrossX, mCrossY)->getType()
- == MapItemType::CROSS)
+ if ((mCrossX || mCrossY) &&
+ layer->getTile(mCrossX, mCrossY) &&
+ layer->getTile(mCrossX, mCrossY)->getType() == MapItemType::CROSS)
{
layer->setTile(mCrossX, mCrossY, MapItemType::EMPTY);
}
@@ -2502,7 +2502,7 @@ void LocalPlayer::setRealPos(const int x, const int y)
if (!mapItem || mapItem->getType() == MapItemType::EMPTY)
{
- if (getTileX() != x && getTileY() != y)
+ if (mX != x && mY != y)
layer->setTile(x, y, MapItemType::CROSS);
}
}
@@ -2531,8 +2531,8 @@ void LocalPlayer::fixAttackTarget()
const Path debugPath = mMap->findPath(
(mPixelX - mapTileSize / 2) / mapTileSize,
(mPixelY - mapTileSize) / mapTileSize,
- mTarget->getTileX(),
- mTarget->getTileY(),
+ mTarget->mX,
+ mTarget->mY,
getBlockWalkMask(),
0);
diff --git a/src/being/localplayer.h b/src/being/localplayer.h
index b062cd6c2..7244664fb 100644
--- a/src/being/localplayer.h
+++ b/src/being/localplayer.h
@@ -391,10 +391,10 @@ class LocalPlayer final : public Being,
const bool updateHash = true);
int getLastAttackX() const override final
- { return mTarget ? mTarget->getTileX() : mLastAttackX; }
+ { return mTarget ? mTarget->mX : mLastAttackX; }
int getLastAttackY() const override final
- { return mTarget ? mTarget->getTileY() : mLastAttackY; }
+ { return mTarget ? mTarget->mY : mLastAttackY; }
void attributeChanged(const AttributesT id,
const int oldVal,