straightforward signal and slots system for lua events

modules/console/console.lua

@@ -67,7 +67,7 @@ end
 function Console.init()
   consoleWidget = UI.loadAndDisplay("/console/console.otui")
   consoleWidget:hide()
-  consoleWidget.onKeyPress = onKeyPress
+  connect(consoleWidget, { onKeyPress = onKeyPress })
 
   commandLineEdit = consoleWidget:getChildById('commandLineEdit')
   consoleBuffer = consoleWidget:getChildById('consoleBuffer')

modules/core/util.lua

@@ -11,3 +11,16 @@ function createEnvironment()
   setmetatable(env, { __index = _G} )
   return env
 end
+
+function connect(object, signalsAndSlots)
+  for signal,slot in pairs(signalsAndSlots) do
+    if not object[signal] then
+        object[signal] = slot
+    elseif type(object[signal]) == 'function' then
+        object[signal] = { object[signal], slot }
+    elseif type(signal) == 'table' then
+        table.insert(object[signal], slot)
+    else
+    end
+  end
+end

modules/game/game.lua

@@ -7,6 +7,9 @@ local function onGameKeyPress(self, keyCode, keyChar, keyboardModifiers)
     if keyCode == KeyG then
       CharacterList.show()
       return true
+    elseif keyCode == KeyQ then
+      Game.logout()
+      return true
     end
   end
   return false

src/framework/luascript/luainterface.cpp

@@ -419,21 +419,32 @@ int LuaInterface::protectedCall(int numArgs, int requestedResults)
         else if(isTable(funcIndex)) {
             // loop through table values
             pushNil();
+            bool done = false;
             while(next(funcIndex-1)) {
                 if(isFunction()) {
                     // repush arguments
                     for(int i=0;i<numArgs;++i)
-                        pushValue(-(numArgs-i)-2);
+                        pushValue(-numArgs-2);
 
                     int rets = safeCall(numArgs);
-                    pop(rets);
+                    if(rets == 1) {
-                }
+                        done = popBoolean();
-                // just ignore
+                        if(done) {
-                else {
+                            pop();
-                    pop();
+                            break;
+                        }
+                    } else if(rets != 0)
+                        throw LuaException("function call didn't return the expected number of results", 0);
+                } else {
+                    throw LuaException("attempt to call a non function", 0);
                 }
             }
             pop(numArgs + 1); // pops the table of function and arguments
+
+            if(requestedResults == -1 || requestedResults == 1) {
+                numRets = 1;
+                pushBoolean(done);
+            }
         }
         // nil values are ignored
         else if(isNil(funcIndex)) {

src/otclient/core/game.cpp

@@ -62,13 +62,15 @@ void Game::processConnectionError(const boost::system::error_code& error)
 {
     // connection errors only have meaning if we still have a protocol
     if(m_protocolGame) {
-        g_lua.callGlobalField("Game", "onConnectionError", error.message());
+        if(error != asio::error::eof)
+            g_lua.callGlobalField("Game", "onConnectionError", error.message());
 
-        if(m_online)
+        if(m_online) {
             processLogout();
-
+        } else {
-        // disconnect isn't needed, we are already disconnected
+            m_protocolGame->disconnect();
-        m_protocolGame.reset();
+            m_protocolGame.reset();
+        }
     }
 }