Răsfoiți Sursa

* small memory leak fixed
* minor changes

mateuszb 16 ani în urmă
părinte
comite
d17469bc7a
3 a modificat fișierele cu 13 adăugiri și 50 ștergeri
  1. 6 42
      client/CBattleInterface.cpp
  2. 7 7
      client/CCreatureAnimation.cpp
  3. 0 1
      client/CPlayerInterface.cpp

+ 6 - 42
client/CBattleInterface.cpp

@@ -2009,54 +2009,18 @@ void CBattleInterface::attackingShowHelper()
 				// TODO: see comment above
 				if (attackingInfo->sh == -1)
 					attackingInfo->sh = CGI->soundh->playSound(aStack.creature->sounds.attack);
+
+				std::map<int, int> dirToType = boost::assign::map_list_of (0, 11)(1, 11)(2, 12)(3, 13)(4, 13)(5, 12);
+				int type; //dependent on attack direction
 				if(aStack.creature->isDoubleWide())
 				{
-					switch(BattleInfo::mutualPosition(aStack.position+attackingInfo->posShiftDueToDist, attackingInfo->dest)) //attack direction
-					{
-						case 0:
-							creAnims[attackingInfo->ID]->setType(11);
-							break;
-						case 1:
-							creAnims[attackingInfo->ID]->setType(11);
-							break;
-						case 2:
-							creAnims[attackingInfo->ID]->setType(12);
-							break;
-						case 3:
-							creAnims[attackingInfo->ID]->setType(13);
-							break;
-						case 4:
-							creAnims[attackingInfo->ID]->setType(13);
-							break;
-						case 5:
-							creAnims[attackingInfo->ID]->setType(12);
-							break;
-					}
+					type = dirToType[ BattleInfo::mutualPosition(aStack.position + attackingInfo->posShiftDueToDist, attackingInfo->dest) ]; //attack direction
 				}
 				else //else for if(aStack.creature->isDoubleWide())
 				{
-					switch(BattleInfo::mutualPosition(aStack.position, attackingInfo->dest)) //attack direction
-					{
-						case 0:
-							creAnims[attackingInfo->ID]->setType(11);
-							break;
-						case 1:
-							creAnims[attackingInfo->ID]->setType(11);
-							break;
-						case 2:
-							creAnims[attackingInfo->ID]->setType(12);
-							break;
-						case 3:
-							creAnims[attackingInfo->ID]->setType(13);
-							break;
-						case 4:
-							creAnims[attackingInfo->ID]->setType(13);
-							break;
-						case 5:
-							creAnims[attackingInfo->ID]->setType(12);
-							break;
-					}
+					type = BattleInfo::mutualPosition(aStack.position, attackingInfo->dest);
 				}
+				creAnims[attackingInfo->ID]->setType(type);
 			}
 		}
 		else if(attackingInfo->frame == (attackingInfo->maxframe - 1))

+ 7 - 7
client/CCreatureAnimation.cpp

@@ -283,13 +283,13 @@ CCreatureAnimation::~CCreatureAnimation()
 }
 
 inline void CCreatureAnimation::putPixel(
-        SDL_Surface * dest,
-        const int & ftcp,
-        const BMPPalette & color,
-        const unsigned char & palc,
-        const bool & yellowBorder,
-		const bool & blueBorder,
-		const unsigned char & animCount
+	SDL_Surface * dest,
+	const int & ftcp,
+	const BMPPalette & color,
+	const unsigned char & palc,
+	const bool & yellowBorder,
+	const bool & blueBorder,
+	const unsigned char & animCount
 ) const
 {	
     if(palc!=0)

+ 0 - 1
client/CPlayerInterface.cpp

@@ -1719,7 +1719,6 @@ void CPlayerInterface::objectPropertyChanged(const SetObjectProperty * sop)
 	//redraw minimap if owner changed
 	if(sop->what == 1)
 	{
-		adventureInt->minimap.initFlaggableObjs();
 		const CGObjectInstance * obj = cb->getObjectInfo(sop->id);
 		std::set<int3> pos = obj->getBlockedPos();
 		for(std::set<int3>::const_iterator it = pos.begin(); it != pos.end(); ++it)