auto frameskip by Iulius. the automatic should make it so that dolphin doesn't skip more then needed (example: you have frameskip 9 but only need 4.2, auto will make it skip on 4.2)

framelimit should be on

git-svn-id: https://dolphin-emu.googlecode.com/svn/trunk@3972 8ced0084-cf51-0410-be5f-012b33b47a6e

Source/Core/Core/Src/Core.cpp

@@ -67,6 +67,8 @@ namespace Core
  
 
 // Declarations and definitions
+Common::Timer Timer;
+u32 frames = 0;
 
  
 // Function forwarding
@@ -587,6 +589,30 @@ void Callback_VideoLog(const TCHAR *_szMessage, int _bDoBreak)
 	INFO_LOG(VIDEO, _szMessage);
 }
  
+// reports if a frame should be skipped or not
+// depending on the framelimit set
+bool report_slow(int skipped)
+{
+	u32 targetfps = SConfig::GetInstance().m_Framelimit * 5;
+	double wait_frametime;
+
+	if (targetfps < 5)
+		wait_frametime = (1000.0 / VideoInterface::TargetRefreshRate);
+	else
+		wait_frametime = (1000.0 / targetfps);
+
+	bool fps_slow;
+
+	if (Timer.GetTimeDifference() < wait_frametime * (frames + skipped - 1))
+		fps_slow=false;
+	else
+		fps_slow=true;
+
+	if (targetfps == 5)
+		fps_slow=true;
+
+	return fps_slow;
+}
 
 // Callback_VideoCopiedToXFB
 // WARNING - THIS IS EXECUTED FROM VIDEO THREAD
@@ -599,15 +625,18 @@ void Callback_VideoCopiedToXFB(bool video_update)
     SCoreStartupParameter& _CoreParameter = SConfig::GetInstance().m_LocalCoreStartupParameter;
 	
 	//count FPS and VPS
-	static Common::Timer Timer;
-	static u32 frames = 0;
 	static u32 videoupd = 0;
+	static u32 no_framelimit = 0;
+	
 
 	if (video_update)
 		videoupd++;
 	else
 		frames++;
 
+	if (no_framelimit>0)
+		no_framelimit--;
+
 	// Custom frame limiter
 	// ¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯
 	u32 targetfps = SConfig::GetInstance().m_Framelimit * 5;
@@ -616,18 +645,28 @@ void Callback_VideoCopiedToXFB(bool video_update)
 	{
 		double wait_frametime = (1000.0 / targetfps);
 
+		if (Timer.GetTimeDifference() >= wait_frametime * frames)
+			no_framelimit=Timer.GetTimeDifference();
+
 		while (Timer.GetTimeDifference() < wait_frametime * frames)
-			Common::SleepCurrentThread(1);
+		{
+			if (no_framelimit==0)
+				Common::SleepCurrentThread(1);
+		}
 	}
 	else if (targetfps < 5)
 	{
 		double wait_frametime = (1000.0 / VideoInterface::TargetRefreshRate);
 
+		if (Timer.GetTimeDifference() >= wait_frametime * frames)
+			no_framelimit=Timer.GetTimeDifference();
+
 		while (Timer.GetTimeDifference() < wait_frametime * videoupd)
 		{
 			// TODO : This is wrong, the sleep shouldn't be there but rather in cputhread
 			// as it's not based on the fps but on the refresh rate...
-			Common::SleepCurrentThread(1);
+			if (no_framelimit==0)
+				Common::SleepCurrentThread(1);
 		}
 	}
 

Source/Core/Core/Src/Core.h

@@ -72,6 +72,8 @@ namespace Core
     void SetBlockStart(u32 addr);
     void StopTrace();
 
+	bool report_slow(int skipped);
+
 	// -----------------------------------------
 	#ifdef RERECORDING
 	// -----------------

Source/Core/Core/Src/OnFrame.cpp

@@ -33,17 +33,16 @@ bool g_bFirstKey = true;
 int g_framesToSkip = 0, g_frameSkipCounter = 0;
 
 void FrameUpdate() {
+
+
 	if (g_bFrameStep)
 		Core::SetState(Core::CORE_PAUSE);
 
+	FrameSkipping();
+
 	if (g_bAutoFire)
 		g_bFirstKey = !g_bFirstKey;
 	
-	if (g_framesToSkip)
-		FrameSkipping();
-	else
-		CPluginManager::GetInstance().GetVideo()->Video_SetRendering(true);
-	
 }
 
 void SetFrameSkipping(unsigned int framesToSkip) {
@@ -123,7 +122,7 @@ void FrameSkipping()
 	cs_frameSkip.Enter();
 
 	g_frameSkipCounter++;
-	if (g_frameSkipCounter > g_framesToSkip)
+	if (g_frameSkipCounter > g_framesToSkip || Core::report_slow(g_frameSkipCounter) == false)
 		g_frameSkipCounter = 0;
 
 	CPluginManager::GetInstance().GetVideo()->Video_SetRendering(!g_frameSkipCounter);