change way animations keys are handled (thanks KittyCat!)

apps/openmw/mwmechanics/character.cpp

@@ -238,7 +238,7 @@ void CharacterController::update(float duration, Movement &movement)
     }
     else if(!cls.getCreatureStats(mPtr).isDead())
     {
-        std::list<std::pair<std::string,std::string>> lastKeys = cls.getCreatureStats(mPtr).getLastAnimKey();
+        /*std::list<std::pair<std::string,std::string>> lastKeys = cls.getCreatureStats(mPtr).getLastAnimKey();
         for(std::list<std::pair<std::string,std::string>>::iterator it = cls.getCreatureStats(mPtr).getLastAnimKey().begin();
             it!=cls.getCreatureStats(mPtr).getLastAnimKey().end();)
         {
@@ -256,7 +256,7 @@ void CharacterController::update(float duration, Movement &movement)
                 if(lastKey.first.compare(off, len, "chop large follow stop") == 0){ mUpperBodyState = UpperCharState_WeapEquiped;std::cout << "FINISHED";}
             }
             it = cls.getCreatureStats(mPtr).getLastAnimKey().erase(it);
-        }
+        }*/
 
         MWBase::World *world = MWBase::Environment::get().getWorld();
 
@@ -478,6 +478,7 @@ void CharacterController::update(float duration, Movement &movement)
                 if(mUpperBodyState == UpperCharState_WeapEquiped)
                 {
                     std::string weapgroup;
+                    std::cout << "attaquing";
                     getWeaponGroup(mWeaponType, weapgroup);
                     mAnimation->play(weapgroup, Priority_Weapon,
                             MWRender::Animation::Group_UpperBody, true,
@@ -490,6 +491,16 @@ void CharacterController::update(float duration, Movement &movement)
                 mUpperBodyState = UpperCharState_ChopReadying;
             }
 
+            std::string weapgroup;
+            getWeaponGroup(mWeaponType, weapgroup);
+            std::string start;
+            std::string stop;
+            float complete;
+            bool animPlaying = mAnimation->getInfo(weapgroup,&complete,&start,&stop);
+            if(mUpperBodyState == UpperCharState_EquipingWeap && !animPlaying) mUpperBodyState = UpperCharState_WeapEquiped;
+            if(mUpperBodyState == UpperCharState_UnEquipingWeap && !animPlaying) mUpperBodyState = UpperCharState_Nothing;
+            if(mUpperBodyState == UpperCharState_ChopReadying && !animPlaying) mUpperBodyState = UpperCharState_WeapEquiped;
+
             MWWorld::ContainerStoreIterator torch = inv.getSlot(MWWorld::InventoryStore::Slot_CarriedLeft);
             if(torch != inv.end() && torch->getTypeName() == typeid(ESM::Light).name())
             {

apps/openmw/mwmechanics/creaturestats.cpp

@@ -96,11 +96,6 @@ namespace MWMechanics
         return mAttackingOrSpell;
     }
 
-    std::list<std::pair<std::string,std::string>> &CreatureStats::getLastAnimKey()
-    {
-        return mLastAnimKeys;
-    }
-
     int CreatureStats::getLevel() const
     {
         return mLevel;
@@ -207,12 +202,6 @@ namespace MWMechanics
         mAttackingOrSpell = attackingOrSpell;
     }
 
-    /*void CreatureStats::setLastAnimKey(std::string key,std::string animGroup)
-    {
-        mLastAnimKey = key;
-        mLastAnimGroup = animGroup;
-    }*/
-
     void CreatureStats::setAiSetting (int index, int value)
     {
         assert (index>=0 && index<4);

apps/openmw/mwmechanics/creaturestats.hpp

@@ -35,7 +35,6 @@ namespace MWMechanics
         bool mAttacked;
         bool mHostile;
         bool mAttackingOrSpell;//for the player, this is true if the left mouse button is pressed, false if not.
-        std::list<std::pair<std::string,std::string>> mLastAnimKeys;
 
     public:
         CreatureStats();
@@ -58,8 +57,6 @@ namespace MWMechanics
 
         const bool & getAttackingOrSpell() const;
 
-        std::list<std::pair<std::string,std::string>> & getLastAnimKey();
-
         int getLevel() const;
 
         int getAiSetting (int index) const;
@@ -91,8 +88,6 @@ namespace MWMechanics
 
         void setAttackingOrSpell(const bool &attackingOrSpell);
 
-        //void addLastAnimKey(std::string key,std::string animGroup);
-
         void setLevel(int level);
 
         void setAiSetting (int index, int value);

apps/openmw/mwrender/animation.cpp

@@ -15,11 +15,6 @@
 
 #include "../mwmechanics/character.hpp"
 
-
-//TODO: remove that and create a class interface
-#include "../mwmechanics/creaturestats.hpp"
-#include "../mwworld/class.hpp"
-
 namespace MWRender
 {
 
@@ -480,8 +475,6 @@ bool Animation::handleTextKey(AnimState &state, const std::string &groupname, co
         return true;
     }
 
-    MWWorld::Class::get(mPtr).getCreatureStats(mPtr).getLastAnimKey().push_back(std::pair<std::string,std::string>(key->second,groupname));
-
     if(evt.compare(off, len, "equip attach") == 0)
     {
         showWeapons(true);