diff options
| author | Bjørn Lindeijer <bjorn@lindeijer.nl> | 2006-11-30 21:01:26 +0000 |
|---|---|---|
| committer | Bjørn Lindeijer <bjorn@lindeijer.nl> | 2006-11-30 21:01:26 +0000 |
| commit | 30aa32025d3406920330390b97001ade27e4343a (patch) | |
| tree | e3512dc59c0513c090b3c1af2f3d24e92dae9255 /src/player.cpp | |
| parent | c24f9a6ec94e42ae47546c3e93794abb2956d3c1 (diff) | |
| download | mana-30aa32025d3406920330390b97001ade27e4343a.tar.gz mana-30aa32025d3406920330390b97001ade27e4343a.tar.bz2 mana-30aa32025d3406920330390b97001ade27e4343a.tar.xz mana-30aa32025d3406920330390b97001ade27e4343a.zip | |
Some optimizations to setSex, setWeapon and handling of player update messages.
Diffstat (limited to 'src/player.cpp')
| -rw-r--r-- | src/player.cpp | 31 |
1 files changed, 18 insertions, 13 deletions
diff --git a/src/player.cpp b/src/player.cpp index cd6986b3..533d88fa 100644 --- a/src/player.cpp +++ b/src/player.cpp @@ -89,28 +89,31 @@ Player::setSex(Uint8 sex) { if (sex != mSex) { - //reload base sprite - delete mSprites[BASE_SPRITE]; + // Reload base sprite + AnimatedSprite *newBaseSprite; if (sex == 0) { - mSprites[BASE_SPRITE] = new AnimatedSprite( + newBaseSprite = new AnimatedSprite( "graphics/sprites/player_male_base.xml"); } else { - mSprites[BASE_SPRITE] = new AnimatedSprite( + newBaseSprite = new AnimatedSprite( "graphics/sprites/player_female_base.xml"); } + delete mSprites[BASE_SPRITE]; + mSprites[BASE_SPRITE] = newBaseSprite; - //reload equipment + // Reload equipment for (int i=1; i<VECTOREND_SPRITE ; i++) { if (i != HAIR_SPRITE && mEquipmentSpriteIDs.at(i) != 0) { - delete mSprites[i]; - mSprites[i] = new AnimatedSprite( + AnimatedSprite *newEqSprite = new AnimatedSprite( "graphics/sprites/" + EquipmentDB::get( mEquipmentSpriteIDs.at(i))->getSprite(sex)); + delete mSprites[i]; + mSprites[i] = newEqSprite; } } } @@ -124,28 +127,30 @@ Player::setWeapon(Uint16 weapon) { if (weapon != mWeapon) { - delete mSprites[WEAPON_SPRITE]; - mSprites[WEAPON_SPRITE] = NULL; + AnimatedSprite *newWeaponSprite = NULL; switch (weapon) { case 0: - mSprites[WEAPON_SPRITE] = + newWeaponSprite = new AnimatedSprite("graphics/sprites/weapon-fist.xml"); break; case 1: - mSprites[WEAPON_SPRITE] = + newWeaponSprite = new AnimatedSprite("graphics/sprites/weapon-dagger.xml"); break; case 2: - mSprites[WEAPON_SPRITE] = + newWeaponSprite = new AnimatedSprite("graphics/sprites/weapon-bow.xml"); break; case 3: - mSprites[WEAPON_SPRITE] = + newWeaponSprite = new AnimatedSprite("graphics/sprites/weapon-scythe.xml"); break; } + + delete mSprites[WEAPON_SPRITE]; + mSprites[WEAPON_SPRITE] = newWeaponSprite; } Being::setWeapon(weapon); } |