SDL: Remove unnecessary sleeps in the SDLMain loop (we now handle it …

…in NativeFrame)
hrydgard · Jan 27, 2024 · cb89dff · cb89dff
1 parent 437dbe4
commit cb89dff
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 15 deletions.
diff --git a/SDL/SDLMain.cpp b/SDL/SDLMain.cpp
@@ -675,17 +675,7 @@ static void EmuThreadFunc(GraphicsContext *graphicsContext) {
 	NativeInitGraphics(graphicsContext);
 
 	while (emuThreadState != (int)EmuThreadState::QUIT_REQUESTED) {
-		double startTime = time_now_d();
-
 		UpdateRunLoop(graphicsContext);
-
-		// Simple throttling to not burn the GPU in the menu.
-		if (GetUIState() != UISTATE_INGAME || !PSP_IsInited()) {
-			double diffTime = time_now_d() - startTime;
-			int sleepTime = (int)(1000.0 / 60.0) - (int)(diffTime * 1000.0);
-			if (sleepTime > 0)
-				sleep_ms(sleepTime);
-		}
 	}
 	emuThreadState = (int)EmuThreadState::STOPPED;
 	graphicsContext->StopThread();

diff --git a/UI/NativeApp.cpp b/UI/NativeApp.cpp
@@ -1176,20 +1176,20 @@ void NativeFrame(GraphicsContext *graphicsContext) {
 		graphicsContext->Poll();
 	}
 
+	SendMouseDeltaAxis();
+
 	if (!(renderFlags & ScreenRenderFlags::HANDLED_THROTTLING)) {
+		// TODO: We should ideally mix this with game audio.
+		g_BackgroundAudio.Play();
+
 		float refreshRate = System_GetPropertyFloat(SYSPROP_DISPLAY_REFRESH_RATE);
 		// Simple throttling to not burn the GPU in the menu.
 		// TODO: This should move into NativeFrame. Also, it's only necessary in MAILBOX or IMMEDIATE presentation modes.
 		double diffTime = time_now_d() - startTime;
 		int sleepTime = (int)(1000.0 / refreshRate) - (int)(diffTime * 1000.0);
 		if (sleepTime > 0)
 			sleep_ms(sleepTime);
-
-		// TODO: We should ideally mix this with game audio.
-		g_BackgroundAudio.Play();
 	}
-
-	SendMouseDeltaAxis();
 }
 
 bool HandleGlobalMessage(UIMessage message, const std::string &value) {