bugfix(gameengine): Fix logic in GameEngine::canUpdateRegularGameLogic()

[xezon]

• Follow up for bugfix(fps): Fix and improve logic time step and render update decoupling #1528

This change fixes a logic error in GameEngine::canUpdateRegularGameLogic. It was spotted when loading a save game while max fps was uncapped, but after saveload it gets capped with TheGlobalData->m_useFpsLimit = true, which was then ignored in GameEngine::canUpdateRegularGameLogic. The result was an update timing discrepancy in GameEngine update, which caused glitches on game world visuals.

TODO

• Replicate in Generals

[greptile-apps]

Greptile Summary

This PR fixes a timing discrepancy bug in GameEngine::canUpdateRegularGameLogic() where changes to TheGlobalData->m_useFpsLimit (e.g. after loading a save game) were not being respected, causing visual glitches.

• Replaces raw getFramesPerSecondLimit() / getLogicTimeScaleFps() calls with getActualFramesPerSecondLimit() / getActualLogicTimeScaleFps(flags), which correctly account for m_useFpsLimit, the enabled state, and game/time-halt flags.
• Removes the now-redundant !enabled shortcut, since getActualLogicTimeScaleFps() already returns RenderFpsPreset::UncappedFpsValue when the logic time scale is disabled, naturally satisfying logicTimeScaleFps >= maxRenderFps.
• Propagates FramePacer::IgnoreFrozenTime | FramePacer::IgnoreHaltedGame flags through the call chain so frozen/halted state is handled in the caller (update()), not baked into the scheduling decision.

Confidence Score: 5/5

The change is a tightly-scoped fix that correctly threads the fps-limit enabled state into the logic update decision. No behavioral regressions are introduced for the network or fast-mode paths.

Both getActualFramesPerSecondLimit() and getActualLogicTimeScaleFps(flags) correctly consolidate the state that was previously read piecemeal, removing the original inconsistency. The dropped !enabled guard is fully covered by getActualLogicTimeScaleFps returning UncappedFpsValue when the scale is disabled, preserving identical runtime behaviour in that branch.

No files require special attention; both changed files are straightforward.

Important Files Changed

Filename	Overview
GeneralsMD/Code/GameEngine/Source/Common/GameEngine.cpp	Core bugfix: replaces raw FramePacer getters with getActual* variants in canUpdateRegularGameLogic and threads logicTimeQueryFlags through canUpdateGameLogic so fps-limit and enable-state changes are respected at runtime.
GeneralsMD/Code/GameEngine/Include/Common/GameEngine.h	Header updated to match the new UnsignedInt logicTimeQueryFlags parameter signatures for canUpdateGameLogic and canUpdateRegularGameLogic.

_{Reviews (1): Last reviewed commit: "bugfix(gameengine): Fix logic in GameEng..." | Re-trigger Greptile}