updated depencies and increased communication speed a bit

Author: mkalkbrenner

platforms/config.sh

@@ -4,9 +4,9 @@ set -e
 
 SDL_SHA=a8589a84226a6202831a3d49ff4edda4acab9acd
 SDL_IMAGE_SHA=11154afb7855293159588b245b446a4ef09e574f
-PINMAME_SHA=518ebdae4483a654eb7d292469e7d21d25ffacc9
-LIBPPUC_SHA=db26ec9233a84f01246562e3c03718f1d649acec
-LIBDMDUTIL_SHA=cc1d3f663516458c427acdf4aa07c56192a67be3
+PINMAME_SHA=2346ed9951b33e83f3bdd46338a5d9cc4707757c
+LIBPPUC_SHA=61651c190235e1d0d00f6e7f1e6c7b95fea61e04
+LIBDMDUTIL_SHA=5c916d53f5ff6535a1403f6734218550ca2a7e87
 
 
 if [ -z "${BUILD_TYPE}" ]; then

src/ppuc.cpp

@@ -28,6 +28,8 @@
 #include "io-boards/Event.h"
 #include "libpinmame.h"
 
+#define MAIN_LOOP_SLEEP_US 20  // Main loop sleep time in microseconds
+
 SDL_AudioStream* m_pstream = nullptr;
 SDL_AudioSpec audioSpec;
 
@@ -902,44 +904,33 @@ int main(int argc, char** argv)
 
   if (PinmameRun(opt_rom) == PINMAME_STATUS_OK)
   {
-    // Pinball machines were slower than modern CPUs. There's no need to
-    // update states too frequently at full speed.
-    int sleep_us = 1000;
-    // Poll I/O boards for events (mainly switches) every 50us.
-    int poll_interval_ms = 50;
-    int poll_trigger = poll_interval_ms * 1000 / sleep_us;
     int index_recv = 0;
 
     ppuc->StartUpdates();
 
     while (running)
     {
-      std::this_thread::sleep_for(std::chrono::microseconds(sleep_us));
+      std::this_thread::sleep_for(std::chrono::microseconds(MAIN_LOOP_SLEEP_US));
 
       if (!game_state)
       {
         continue;
       }
 
-      if (--poll_trigger <= 0)
+      PPUCSwitchState* switchState;
+      while ((switchState = ppuc->GetNextSwitchState()) != nullptr)
       {
-        poll_trigger = poll_interval_ms * 1000 / sleep_us;
-
-        PPUCSwitchState* switchState;
-        while ((switchState = ppuc->GetNextSwitchState()) != nullptr)
+        if (opt_debug || opt_debug_switches)
         {
-          if (opt_debug || opt_debug_switches)
-          {
-            printf("Switch updated: #%d, %d\n", switchState->number, switchState->state);
-          }
+          printf("Switch updated: #%d, %d\n", switchState->number, switchState->state);
+        }
 
-          // Switches between 200 and 240 are custom switches within the io-boards which should not be sent to
-          // pinmame. Switches above 240 will become negative values, for example 243 => -3.
-          if (switchState->number < 200 || switchState->number > 241)
-          {
-            int switchNumber = (switchState->number < 241) ? switchState->number : 240 - switchState->number;
-            PinmameSetSwitch(switchNumber, switchState->state);
-          }
+        // Switches between 200 and 240 are custom switches within the io-boards which should not be sent to
+        // pinmame. Switches above 240 will become negative values, for example 243 => -3.
+        if (switchState->number < 200 || switchState->number > 241)
+        {
+          int switchNumber = (switchState->number < 241) ? switchState->number : 240 - switchState->number;
+          PinmameSetSwitch(switchNumber, switchState->state);
         }
       }
 
@@ -957,7 +948,7 @@ int main(int argc, char** argv)
         ppuc->SetLampState(lampNo, lampState);
       }
 
-      {
+      {  // Needs to be a separate scope for the lock_guard
         // Process any pending render requests
         std::lock_guard<std::mutex> lock(renderMutex);
         while (!renderQueue.empty())