diff options
author | Bjørn Lindeijer <bjorn@lindeijer.nl> | 2009-02-18 20:36:37 +0100 |
---|---|---|
committer | Bjørn Lindeijer <bjorn@lindeijer.nl> | 2009-02-18 20:36:37 +0100 |
commit | faa1a1d9d24008fee298ca688d76a684af96d0a2 (patch) | |
tree | 29a15514f3d96fc392ece03f10df8f68d0d7129f | |
parent | 7e43b82e3b7cda034fab34c15ecfaa97c1a99146 (diff) | |
parent | 218303948fa3aa3fd139b47e303ed1f7c7cfd802 (diff) | |
download | mana-faa1a1d9d24008fee298ca688d76a684af96d0a2.tar.gz mana-faa1a1d9d24008fee298ca688d76a684af96d0a2.tar.bz2 mana-faa1a1d9d24008fee298ca688d76a684af96d0a2.tar.xz mana-faa1a1d9d24008fee298ca688d76a684af96d0a2.zip |
Merge commit 'aethyra/master'
Conflicts:
src/net/npchandler.cpp
-rw-r--r-- | src/localplayer.cpp | 1 | ||||
-rw-r--r-- | src/player.cpp | 45 |
2 files changed, 21 insertions, 25 deletions
diff --git a/src/localplayer.cpp b/src/localplayer.cpp index 44899069..b9654e63 100644 --- a/src/localplayer.cpp +++ b/src/localplayer.cpp @@ -92,7 +92,6 @@ LocalPlayer::~LocalPlayer() { delete mInventory; delete mStorage; - delete mName; for (int i = Being::TC_SMALL; i < Being::NUM_TC; i++) { diff --git a/src/player.cpp b/src/player.cpp index a2a263b3..f620e637 100644 --- a/src/player.cpp +++ b/src/player.cpp @@ -35,7 +35,7 @@ static const int NAME_Y_OFFSET = 30; Player::Player(int id, int job, Map *map): Being(id, job, map) { - mName = 0; + mName = NULL; } Player::~Player() @@ -45,19 +45,21 @@ Player::~Player() void Player::setName(const std::string &name) { - if (mName == 0) + if (mName == NULL) { if (mIsGM) { mNameColor = 0x009000; - mName = new FlashText("(GM) " + name, mPx + NAME_X_OFFSET, mPy + NAME_Y_OFFSET, - gcn::Graphics::CENTER, gcn::Color(0, 255, 0)); + mName = new FlashText("(GM) " + name, mPx + NAME_X_OFFSET, mPy + + NAME_Y_OFFSET, gcn::Graphics::CENTER, + gcn::Color(0, 255, 0)); } else { mNameColor = 0x202020; mName = new FlashText(name, mPx + NAME_X_OFFSET, mPy + NAME_Y_OFFSET, - gcn::Graphics::CENTER, gcn::Color(255, 255, 255)); + gcn::Graphics::CENTER, + gcn::Color(255, 255, 255)); } Being::setName(name); } @@ -65,7 +67,8 @@ void Player::setName(const std::string &name) void Player::logic() { - switch (mAction) { + switch (mAction) + { case STAND: break; @@ -80,22 +83,25 @@ void Player::logic() case WALK: mFrame = (get_elapsed_time(mWalkTime) * 6) / mWalkSpeed; - if (mFrame >= 6) { + + if (mFrame >= 6) nextStep(); - } + break; case ATTACK: int frames = 4; - if ( mEquippedWeapon - && mEquippedWeapon->getAttackType() == ACTION_ATTACK_BOW) + if (mEquippedWeapon && + mEquippedWeapon->getAttackType() == ACTION_ATTACK_BOW) { frames = 5; } + mFrame = (get_elapsed_time(mWalkTime) * frames) / mAttackSpeed; - if (mFrame >= frames) { + + if (mFrame >= frames) nextStep(); - } + break; } @@ -110,9 +116,7 @@ Being::Type Player::getType() const void Player::flash(int time) { if (mName) - { mName->flash(time); - } } void Player::setGender(Gender gender) @@ -131,9 +135,7 @@ void Player::setGender(Gender gender) for (int i = 1; i < VECTOREND_SPRITE; i++) { if (mSpriteIDs.at(i) != 0) - { setSprite(i, mSpriteIDs.at(i), mSpriteColors.at(i)); - } } } } @@ -160,9 +162,7 @@ void Player::setSprite(int slot, int id, std::string color) mSprites[slot] = NULL; if (slot == WEAPON_SPRITE) - { mEquippedWeapon = NULL; - } } else { @@ -173,8 +173,9 @@ void Player::setSprite(int slot, int id, std::string color) { if (!color.empty()) filename += "|" + color; - equipmentSprite = AnimatedSprite::load( - "graphics/sprites/" + filename); + + equipmentSprite = AnimatedSprite::load("graphics/sprites/" + + filename); } if (equipmentSprite) @@ -184,9 +185,7 @@ void Player::setSprite(int slot, int id, std::string color) mSprites[slot] = equipmentSprite; if (slot == WEAPON_SPRITE) - { mEquippedWeapon = &ItemDB::get(id); - } setAction(mAction); } @@ -197,8 +196,6 @@ void Player::setSprite(int slot, int id, std::string color) void Player::updateCoords() { if (mName) - { mName->adviseXY(mPx + NAME_X_OFFSET, mPy + NAME_Y_OFFSET); - } } |