Fix #4591

Author: SuuperW

src/BizHawk.Client.EmuHawk/MainForm.cs

@@ -1076,7 +1076,7 @@ protected override void Dispose(bool disposing)
 		private bool _emulatorPaused;
 		public bool EmulatorPaused
 		{
-			get => _emulatorPaused;
+			get => _emulatorPaused && !_unpauseByFrameAdvance;
 
 			private set
 			{
@@ -1088,6 +1088,22 @@ private set
 			}
 		}
 
+		private bool _unpauseByFrameAdvance;
+		/// <summary>
+		/// Avoids using EmulatorPaused to handle frame advance, thus allowing Lua to unpause during a frame advance.
+		/// </summary>
+		private bool UnpauseByFrameAdvance
+		{
+			get => _unpauseByFrameAdvance;
+			set
+			{
+				if (_unpauseByFrameAdvance == value) return;
+				_unpauseByFrameAdvance = value;
+
+				OnPauseToggle(!value);
+			}
+		}
+
 		private void OnPauseToggle(bool newPauseState)
 		{
 			if (!newPauseState) // Unpausing
@@ -2936,21 +2952,22 @@ private void StepRunLoop_Core(bool force = false)
 					// handle the initial trigger of a frame advance
 					runFrame = true;
 					_frameAdvanceTimestamp = currentTimestamp;
+					// Pausing is inconsistent with the behavior of TAStudio while seeking, but it's always been this way so.
 					PauseEmulator();
 				}
 				else if (frameProgressTimeElapsed)
 				{
 					runFrame = true;
 					_runloopFrameProgress = true;
-					UnpauseEmulator();
+					UnpauseByFrameAdvance = true;
 				}
 			}
 			else
 			{
 				if (_runloopFrameAdvance)
 				{
 					// handle release of frame advance
-					PauseEmulator();
+					UnpauseByFrameAdvance = false;
 				}
 				_runloopFrameProgress = false;
 			}