tab duplicity about gemrb HOT 12 CLOSED

diff --git a/gemrb/core/GUI/GameControl.cpp b/gemrb/core/GUI/GameControl.cpp
index 50b1bd7a4..1e655a38f 100644
--- a/gemrb/core/GUI/GameControl.cpp
+++ b/gemrb/core/GUI/GameControl.cpp
@@ -119,6 +119,8 @@ GameControl::GameControl(const Region& frame)
 
        EventMgr::EventCallback* cb = new MethodCallback<GameControl, const Event&, bool>(this, &GameControl::OnGlobalMouseMove);
        EventMgr::RegisterEventMonitor(cb, Event::MouseMoveMask);
+       EventMgr::EventCallback *cb2 = new MethodCallback<GameControl, const Event&, bool>(this, &GameControl::DispatchEvent);
+       EventMgr::RegisterEventMonitor(cb2, Event::KeyDownMask);
 }
 
 //TODO:
@@ -592,6 +594,24 @@ void GameControl::DrawSelf(Region screen, const Region& /*clip*/)
        }
 }
 
+// this existly only so tab can be handled
+// it's used both for tooltips everywhere and hp display on game control
+bool GameControl::DispatchEvent(const Event& event)
+{
+       if (event.keyboard.keycode == GEM_TAB) {
+               Game *game = core->GetGame();
+               if (!game) return false;
+               // show partymember hp/maxhp as overhead text
+               for (int pm=0; pm < game->GetPartySize(false); pm++) {
+                       Actor *pc = game->GetPC(pm, true);
+                       if (!pc) continue;
+                       pc->DisplayHeadHPRatio();
+               }
+               return true;
+       }
+       return false;
+}
+
 /** Key Press Event */
 bool GameControl::OnKeyPress(const KeyboardEvent& Key, unsigned short mod)
 {
@@ -630,11 +650,7 @@ bool GameControl::OnKeyPress(const KeyboardEvent& Key, unsigned short mod)
                        break;
                case GEM_TAB:
                        // show partymember hp/maxhp as overhead text
-                       for (int pm=0; pm < game->GetPartySize(false); pm++) {
-                               Actor *pc = game->GetPC(pm, true);
-                               if (!pc) continue;
-                               pc->DisplayHeadHPRatio();
-                       }
+                       // this is handled in DispatchEvent due to tab having two functions
                        break;
                case GEM_ESCAPE:
                        core->SetEventFlag(EF_ACTION|EF_RESETTARGET);
diff --git a/gemrb/core/GUI/GameControl.h b/gemrb/core/GUI/GameControl.h
index fc80b87d4..10021f6ec 100644
--- a/gemrb/core/GUI/GameControl.h
+++ b/gemrb/core/GUI/GameControl.h
@@ -178,6 +178,8 @@ public:
        /** Mouse Over Event */
        void OnMouseOver(const MouseEvent&);
        void OnMouseDrag(const MouseEvent& /*me*/);
+       /** Currently only deals with the GEM_TAB exception */
+       bool DispatchEvent(const Event& event);
 
        /** Mouse Button Down */
        void OnMouseDown(const MouseEvent& /*me*/, unsigned short Mod);
diff --git a/gemrb/plugins/GUIScript/GUIScript.cpp b/gemrb/plugins/GUIScript/GUIScript.cpp
index 745a1991b..a7f2187ff 100644
--- a/gemrb/plugins/GUIScript/GUIScript.cpp
+++ b/gemrb/plugins/GUIScript/GUIScript.cpp
@@ -13167,6 +13167,7 @@ bool GUIScript::Init(void)
                return false;
        }
        PyRun_SimpleString( "GEMRB_VERSION = '" GEMRB_STRING "'" );
+//     PyModule_AddStringConstant(pGemRB, "GEMRB_VERSION", GEMRB_STRING);
 
        // Detect GameType if it was set to auto
        if (stricmp( core->GameType, "auto" ) == 0) {

EventCallback* cb = it->second.get();
(*cb)(e);