New impl

Author: xezon

GeneralsMD/Code/GameEngine/Include/Common/GameEngine.h

@@ -80,6 +80,7 @@ class GameEngine : public SubsystemInterface
 	virtual Int getFramesPerSecondLimit( void ); ///< Get the max render and engine update fps.
 	Real getUpdateTime(); ///< Get the last engine update delta time in seconds.
 	Real getUpdateFps(); ///< Get the last engine update fps.
+	Real getBaseOverUpdateFpsRatio(Real minUpdateFps = 5.0f); ///< Get the last engine base over update fps ratio. Used to scale user inputs to a frame rate independent speed.
 
 	static Bool isTimeFrozen(); ///< Returns true if a script has frozen time.
 	static Bool isGameHalted(); ///< Returns true if the game is paused or the network is stalling.

GeneralsMD/Code/GameEngine/Source/Common/GameEngine.cpp

@@ -338,6 +338,14 @@ Real GameEngine::getUpdateFps()
 	return 1.0f / m_updateTime;
 }
 
+//-------------------------------------------------------------------------------------------------
+Real GameEngine::getBaseOverUpdateFpsRatio(Real minUpdateFps)
+{
+	// TheSuperHackers @info Update fps is floored to default 5 fps, 200 ms.
+	// Useful to prevent insane ratios on frame spikes/stalls.
+	return (Real)BaseFps / std::max(getUpdateFps(), minUpdateFps);
+}
+
 //-------------------------------------------------------------------------------------------------
 Bool GameEngine::isTimeFrozen()
 {

GeneralsMD/Code/GameEngine/Source/GameClient/InGameUI.cpp

@@ -1945,7 +1945,7 @@ void InGameUI::update( void )
 	if (m_cameraRotatingLeft || m_cameraRotatingRight || m_cameraZoomingIn || m_cameraZoomingOut)
 	{
 		// TheSuperHackers @tweak The camera rotation and zoom are now decoupled from the render update.
-		const Real fpsRatio = (Real)BaseFps / TheGameEngine->getUpdateFps();
+		const Real fpsRatio = TheGameEngine->getBaseOverUpdateFpsRatio();
 		const Real rotateAngle = TheGlobalData->m_keyboardCameraRotateSpeed * fpsRatio;
 		const Real zoomHeight = (Real)View::ZoomHeightPerSecond * fpsRatio;
 

GeneralsMD/Code/GameEngine/Source/GameClient/MessageStream/LookAtXlat.cpp

@@ -439,7 +439,7 @@ GameMessageDisposition LookAtTranslator::translateGameMessage(const GameMessage
 			{
 
 				// TheSuperHackers @bugfix Mauller 07/06/2025 The camera scrolling is now decoupled from the render update.
-				const Real fpsRatio = (Real)BaseFps / TheGameEngine->getUpdateFps();
+				const Real fpsRatio = TheGameEngine->getBaseOverUpdateFpsRatio();
 
 				switch (m_scrollType)
 				{

GeneralsMD/Code/GameEngineDevice/Source/W3DDevice/GameClient/W3DView.cpp

@@ -1355,7 +1355,7 @@ void W3DView::update(void)
 			// if scrolling, only adjust if we're too close or too far
 			if (m_scrollAmount.length() < m_scrollAmountCutoff || (m_currentHeightAboveGround < m_minHeightAboveGround) || (TheGlobalData->m_enforceMaxCameraHeight && m_currentHeightAboveGround > m_maxHeightAboveGround))
 			{
-				const Real fpsRatio = (Real)BaseFps / TheGameEngine->getUpdateFps();
+				const Real fpsRatio = TheGameEngine->getBaseOverUpdateFpsRatio();
 				const Real zoomAdj = (desiredZoom - m_zoom) * TheGlobalData->m_cameraAdjustSpeed * fpsRatio;
 				if (fabs(zoomAdj) >= 0.0001f)	// only do positive
 				{
@@ -1367,7 +1367,7 @@ void W3DView::update(void)
 		else if (!didScriptedMovement)
 		{
 			// we're not scrolling; settle toward desired height above ground
-			const Real fpsRatio = (Real)BaseFps / TheGameEngine->getUpdateFps();
+			const Real fpsRatio = TheGameEngine->getBaseOverUpdateFpsRatio();
 			const Real zoomAdj = (m_zoom - desiredZoom) * TheGlobalData->m_cameraAdjustSpeed * fpsRatio;
 			if (fabs(zoomAdj) >= 0.0001f)
 			{