tweak(fps): Decouple drawable time step from render update (#1451)

Author: xezon

GeneralsMD/Code/GameEngine/Include/GameClient/Display.h

@@ -109,6 +109,7 @@ class Display : public SubsystemInterface
 
 	virtual void drawViews( void );																///< Render all views of the world
 	virtual void updateViews ( void );															///< Updates state of world views
+	virtual void stepViews(); ///< Update views for every fixed time step
 
 	virtual VideoBuffer*	createVideoBuffer( void ) = 0;							///< Create a video buffer that can be used for this display
 

GeneralsMD/Code/GameEngine/Include/GameClient/View.h

@@ -211,6 +211,7 @@ class View : public Snapshot
 
 	virtual void drawView( void ) = 0;															///< Render the world visible in this view.
 	virtual void updateView(void) = 0;					///<called once per frame to determine the final camera and object transforms
+	virtual void stepView() = 0; ///< Update view for every fixed time step
 
 
 

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

@@ -131,6 +131,14 @@ void Display::updateViews( void )
 
 }
 
+void Display::stepViews( void )
+{
+
+	for( View *v = m_viewList; v; v = v->getNextView() )
+		v->stepView();
+
+}
+
 /// Redraw the entire display
 void Display::draw( void )
 {

GeneralsMD/Code/GameEngineDevice/Include/W3DDevice/GameClient/W3DView.h

@@ -140,6 +140,7 @@ class W3DView : public View, public SubsystemInterface
 	virtual void reset( void );
 	virtual void drawView( void );  ///< draw this view
 	virtual void updateView(void);	///<called once per frame to determine the final camera and object transforms
+	virtual void stepView(); ///< Update view for every fixed time step
 
 	virtual void draw( );  ///< draw this view
 	virtual void update( );	///<called once per frame to determine the final camera and object transforms

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

@@ -1726,6 +1726,8 @@ void W3DDisplay::step()
 	}
 
 	WW3D::Sync( syncTime );
+
+	stepViews();
 }
 
 //DECLARE_PERF_TIMER(BigAssRenderLoop)

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

@@ -1080,6 +1080,24 @@ void W3DView::updateView(void)
 	UPDATE();
 }
 
+// TheSuperHackers @tweak xezon 12/08/2025 The drawable update is no longer tied to the
+// render update, but it advanced separately for every fixed time step. This ensures that
+// things like vehicle wheels no longer spin too fast on high frame rates or keep spinning
+// on game pause.
+void W3DView::stepView()
+{
+	if (TheScriptEngine->isTimeFast()) {
+		return; // don't draw - makes it faster :) jba.
+	}
+
+	Region3D axisAlignedRegion;
+	getAxisAlignedViewRegion(axisAlignedRegion);
+
+	// render all of the visible Drawables
+	/// @todo this needs to use a real region partition or something
+	TheGameClient->iterateDrawablesInRegion( &axisAlignedRegion, drawDrawable, this );
+}
+
 //DECLARE_PERF_TIMER(W3DView_updateView)
 void W3DView::update(void)
 {
@@ -1383,14 +1401,6 @@ void W3DView::update(void)
   // Give the terrain a chance to refresh animaing (Seismic) regions, if any.
   TheTerrainVisual->updateSeismicSimulations();
 #endif
-
-	Region3D axisAlignedRegion;
-	getAxisAlignedViewRegion(axisAlignedRegion);
-
-	// render all of the visible Drawables
-	/// @todo this needs to use a real region partition or something
-	if (WW3D::Get_Frame_Time())	//make sure some time actually elapsed
-		TheGameClient->iterateDrawablesInRegion( &axisAlignedRegion, drawDrawable, this );
 }
 
 //-------------------------------------------------------------------------------------------------

GeneralsMD/Code/Tools/WorldBuilder/src/wbview3d.cpp

@@ -183,6 +183,7 @@ class PlaceholderView : public View
 
 	virtual void drawView( void ) {};															///< Render the world visible in this view.
 	virtual void updateView( void ) {};															///< Render the world visible in this view.
+	virtual void stepView() {}; ///< Update view for every fixed time step
 
 	virtual void setZoomLimited( Bool limit ) {}			///< limit the zoom height
 	virtual Bool isZoomLimited( void ) { return TRUE; }							///< get status of zoom limit