TimingEvent: Fix events losing time when rescheduling outside handler

src/core/cdrom.cpp

@@ -523,7 +523,7 @@ TickCount CDROM::GetTicksForSeek() const
 void CDROM::BeginCommand(Command command)
 {
   m_command = command;
-  m_command_event->SetDowncount(GetAckDelayForCommand());
+  m_command_event->Schedule(GetAckDelayForCommand());
   UpdateCommandEvent();
   UpdateStatusRegister();
 }

src/core/timing_event.cpp

@@ -28,27 +28,22 @@ TickCount TimingEvent::GetTicksUntilNextExecution() const
 
 void TimingEvent::Schedule(TickCount ticks)
 {
-  m_downcount = ticks;
-  m_time_since_last_run = 0;
+  const TickCount pending_ticks = m_system->m_cpu->GetPendingTicks();
+  m_downcount = pending_ticks + ticks;
 
-  // Factor in partial time if this was rescheduled outside of an event handler. Say, an MMIO write.
-  if (!m_system->m_running_events)
+  if (!m_active)
   {
-    const TickCount pending_ticks = m_system->m_cpu->GetPendingTicks();
-    m_downcount += pending_ticks;
-    m_time_since_last_run -= pending_ticks;
+    // Event is going active, so we want it to only execute ticks from the current timestamp.
+    m_time_since_last_run = -pending_ticks;
+    m_active = true;
+    m_system->AddActiveEvent(this);
   }
-
-  if (m_active)
+  else
   {
+    // Event is already active, so we leave the time since last run alone, and just modify the downcount.
     // If this is a call from an IO handler for example, re-sort the event queue.
     m_system->SortEvents();
   }
-  else
-  {
-    m_active = true;
-    m_system->AddActiveEvent(this);
-  }
 }
 
 void TimingEvent::SetIntervalAndSchedule(TickCount ticks)
@@ -118,13 +113,3 @@ void TimingEvent::Deactivate()
   m_active = false;
   m_system->RemoveActiveEvent(this);
 }
-
-void TimingEvent::SetDowncount(TickCount downcount)
-{
-  const TickCount pending_ticks = m_system->m_running_events ? 0 : m_system->m_cpu->GetPendingTicks();
-  m_downcount = downcount + pending_ticks;
-  m_time_since_last_run = -pending_ticks;
-
-  if (m_active)
-    m_system->SortEvents();
-}

src/core/timing_event.h

@@ -57,9 +57,6 @@ public:
       Deactivate();
   }
 
-  // Directly alters the downcount of the event.
-  void SetDowncount(TickCount downcount);
-
   // Directly alters the interval of the event.
   void SetInterval(TickCount interval) { m_interval = interval; }
   void SetPeriod(TickCount period) { m_period = period; }