From 0f82a8b4a599fbeb2663680deb406bbc15a67576 Mon Sep 17 00:00:00 2001
From: Andrei Karas <akaras@inbox.ru>
Date: Tue, 17 Apr 2012 22:05:45 +0300
Subject: Move input condition mask to class member variable.

---
 src/inputmanager.cpp | 49 +++++++++++++++++++++++++------------------------
 src/inputmanager.h   | 10 ++++++----
 2 files changed, 31 insertions(+), 28 deletions(-)

diff --git a/src/inputmanager.cpp b/src/inputmanager.cpp
index 51eca37f7..283b07d6c 100644
--- a/src/inputmanager.cpp
+++ b/src/inputmanager.cpp
@@ -63,7 +63,8 @@ class KeyFunctor
 
 InputManager::InputManager() :
     mSetupInput(nullptr),
-    mNewKeyIndex(Input::KEY_NO_VALUE)
+    mNewKeyIndex(Input::KEY_NO_VALUE),
+    mMask(1)
 {
 }
 
@@ -439,6 +440,7 @@ bool InputManager::handleEvent(const SDL_Event &event)
     {
         case SDL_KEYDOWN:
         {
+            updateConditionMask();
             if (handleAssignKey(event, INPUT_KEYBOARD))
                 return true;
 
@@ -465,11 +467,13 @@ bool InputManager::handleEvent(const SDL_Event &event)
         }
         case SDL_KEYUP:
         {
+            updateConditionMask();
             keyboard.handleDeActicateKey(event);
             break;
         }
         case SDL_JOYBUTTONDOWN:
         {
+            updateConditionMask();
 //            joystick.handleActicateButton(event);
             if (handleAssignKey(event, INPUT_JOYSTICK))
                 return true;
@@ -477,6 +481,7 @@ bool InputManager::handleEvent(const SDL_Event &event)
         }
         case SDL_JOYBUTTONUP:
         {
+            updateConditionMask();
 //            joystick.handleDeActicateButton(event);
             break;
         }
@@ -522,65 +527,63 @@ bool InputManager::handleEvent(const SDL_Event &event)
     return false;
 }
 
-int InputManager::getInputConditionMask()
+void InputManager::updateConditionMask()
 {
-    int mask = 1;
+    mMask = 1;
     if (keyboard.isEnabled())
-        mask += COND_ENABLED;
+        mMask += COND_ENABLED;
     if ((!chatWindow || !chatWindow->isInputFocused()) &&
         !NpcDialog::isAnyInputFocused() &&
         !InventoryWindow::isAnyInputFocused() &&
         (!tradeWindow || !tradeWindow->isInpupFocused()))
     {
-        mask += COND_NOINPUT;
+        mMask += COND_NOINPUT;
     }
 
     if (!player_node || !player_node->getAway())
-        mask += COND_NOAWAY;
+        mMask += COND_NOAWAY;
 
     if (!setupWindow || !setupWindow->isVisible())
-        mask += COND_NOSETUP;
+        mMask += COND_NOSETUP;
 
     if (Game::instance()->getValidSpeed())
-        mask += COND_VALIDSPEED;
+        mMask += COND_VALIDSPEED;
 
     if (gui && !gui->getFocusHandler()->getModalFocused())
-        mask += COND_NOMODAL;
+        mMask += COND_NOMODAL;
 
     NpcDialog *dialog = NpcDialog::getActive();
     if (!dialog || !dialog->isTextInputFocused())
-        mask += COND_NONPCINPUT;
+        mMask += COND_NONPCINPUT;
 
     if (!player_node || !player_node->getDisableGameModifiers())
-        mask += COND_EMODS;
+        mMask += COND_EMODS;
 
     if (!isActionActive(Input::KEY_STOP_ATTACK)
         && !isActionActive(Input::KEY_UNTARGET))
     {
-        mask += COND_NOTARGET;
+        mMask += COND_NOTARGET;
     }
 
     if (!player_node || player_node->getFollow().empty())
-        mask += COND_NOFOLLOW;
-
-    return mask;
+        mMask += COND_NOFOLLOW;
 }
 
-bool InputManager::checkKey(const KeyData *key, int mask)
+bool InputManager::checkKey(const KeyData *key)
 {
-//    logger->log("mask=%d, condition=%d", mask, key->condition);
-    if (!key || (key->condition & mask) != key->condition)
+//    logger->log("mask=%d, condition=%d", mMask, key->condition);
+    if (!key || (key->condition & mMask) != key->condition)
         return false;
 
     return (key->modKeyIndex == Input::KEY_NO_VALUE
         || isActionActive(key->modKeyIndex));
 }
 
-bool InputManager::invokeKey(const KeyData *key, int keyNum, int mask)
+bool InputManager::invokeKey(const KeyData *key, int keyNum)
 {
-    if (checkKey(key, mask))
+    if (checkKey(key))
     {
-        InputEvent evt(keyNum, mask);
+        InputEvent evt(keyNum, mMask);
         if ((*(keyData[keyNum].action))(evt))
             return true;
     }
@@ -625,15 +628,13 @@ bool InputManager::triggerAction(const KeysVector *ptrs)
     KeysVectorCIter it = ptrs->begin();
     KeysVectorCIter it_end = ptrs->end();
 
-    int mask = getInputConditionMask();
-
     for (; it != it_end; ++ it)
     {
         const int keyNum = *it;
         if (keyNum < 0 || keyNum >= Input::KEY_TOTAL)
             continue;
 
-        if (invokeKey(&keyData[keyNum], keyNum, mask))
+        if (invokeKey(&keyData[keyNum], keyNum))
             return true;
     }
     return false;
diff --git a/src/inputmanager.h b/src/inputmanager.h
index be0acc23e..b8114d906 100644
--- a/src/inputmanager.h
+++ b/src/inputmanager.h
@@ -88,9 +88,7 @@ class InputManager
 
         bool handleEvent(const SDL_Event &event);
 
-        int getInputConditionMask();
-
-        bool checkKey(const KeyData *key, int mask);
+        bool checkKey(const KeyData *key);
 
         void retrieve();
 
@@ -138,7 +136,7 @@ class InputManager
 
         void updateKeyActionMap(KeyToActionMap &actionMap, int type);
 
-        bool invokeKey(const KeyData *key, int keyNum, int mask);
+        bool invokeKey(const KeyData *key, int keyNum);
 
         bool handleAssignKey(const SDL_Event &event, int type);
 
@@ -148,11 +146,15 @@ class InputManager
 
         void update();
 
+        void updateConditionMask();
+
     protected:
         Setup_Input *mSetupInput;      /**< Reference to setup window */
 
         int mNewKeyIndex;              /**< Index of new key to be assigned */
 
+        int mMask;
+
         KeyFunction mKey[Input::KEY_TOTAL]; /**< Pointer to all the key data */
 };
 
-- 
cgit v1.2.3-60-g2f50