DumbProjectileBehavior and NeutronMissileUpdate logic decouple working

Author: githubawn

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

@@ -101,6 +101,9 @@ class GameEngine : public SubsystemInterface
 
 	Bool m_quitting; ///< true when we need to quit the game
 	Bool m_isActive; ///< app has OS focus.
+
+public:
+	Real getLogicTimeAccumulator() const { return m_logicTimeAccumulator; }
 };
 
 inline void GameEngine::setQuitting( Bool quitting ) { m_quitting = quitting; }

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

@@ -48,6 +48,7 @@ class DrawModule;
 class ClientUpdateModule;
 class View;
 class Locomotor;
+class ProjectileUpdateInterface;
 class Anim2D;
 class Shadow;
 class ModuleInfo;
@@ -652,6 +653,7 @@ class Drawable : public Thing,
 private:
 
 	const Locomotor* getLocomotor() const;
+	void applySubFrameExtrapolation(ProjectileUpdateInterface* pui);
 
 	// note, these are lazily allocated!
 	TintEnvelope*		m_selectionFlashEnvelope;	///< used for selection flash, works WITH m_colorTintEnvelope
@@ -711,6 +713,7 @@ class Drawable : public Thing,
 
 	Matrix3D m_instance;				///< The instance matrix that holds the initial/default position & orientation
 	Real m_instanceScale;				///< the uniform scale factor applied to the instance matrix before it is sent to W3D.
+	Matrix3D m_visualExtrapolationMtx;  ///< Decoupled visual glide matrix
 
 	DrawableInfo				m_drawableInfo;		///< structure pointed to by W3D render objects so they know which drawable they belong to.
 
@@ -730,6 +733,7 @@ class Drawable : public Thing,
 	Bool m_hiddenByStealth;			///< drawable is hidden due to stealth
 	Bool m_instanceIsIdentity;	///< If true, instance matrix can be skipped
 	Bool m_drawableFullyObscuredByShroud;	///<drawable is hidden by shroud/fog
+	Bool m_useExtrapolation;            ///< True if currently gliding
   Bool m_ambientSoundEnabled;
   Bool m_ambientSoundEnabledFromScript;
 

GeneralsMD/Code/GameEngine/Include/GameLogic/Module/DumbProjectileBehavior.h

@@ -94,6 +94,7 @@ class DumbProjectileBehavior : public UpdateModule, public ProjectileUpdateInter
 	virtual ObjectID projectileGetLauncherID() const override { return m_launcherID; }
 	virtual void setFramesTillCountermeasureDiversionOccurs( UnsignedInt frames ) override {}
 	virtual void projectileNowJammed() override {}
+	virtual const Coord3D* getProjectileLogicVelocity() const override { return &m_logicStepVelocity; }
 
 protected:
 
@@ -109,6 +110,7 @@ class DumbProjectileBehavior : public UpdateModule, public ProjectileUpdateInter
 	VecCoord3D						m_flightPath;							///< The frame by frame flight path in a Bezier curve
 	Coord3D								m_flightPathStart;				///< where flight path started (in case we must regen it)
 	Coord3D								m_flightPathEnd;					///< where flight path ends (in case we must regen it)
+	Coord3D								m_logicStepVelocity;			///< Logic frame velocity vector
 	Real									m_flightPathSpeed;				///< flight path speed (in case we must regen it)
 	Int										m_flightPathSegments;			///< number of segments in the flightpath (in case we must regen it)
 	Int										m_currentFlightPathStep;	///< Our current index in the flight path vector.  Quicker than popping off.

GeneralsMD/Code/GameEngine/Include/GameLogic/Module/MissileAIUpdate.h

@@ -96,6 +96,7 @@ class MissileAIUpdate : public AIUpdateInterface, public ProjectileUpdateInterfa
 	virtual ObjectID projectileGetLauncherID() const override { return m_launcherID; }
 	virtual void setFramesTillCountermeasureDiversionOccurs( UnsignedInt frames ) override; ///< Number of frames till missile diverts to countermeasures.
 	virtual void projectileNowJammed() override;///< We lose our Object target and scatter to the ground
+	virtual const Coord3D* getProjectileLogicVelocity() const override { return &m_logicStepVelocity; }
 
 	virtual Bool processCollision(PhysicsBehavior *physics, Object *other) override; ///< Returns true if the physics collide should apply the force.  Normally not.  jba.
 
@@ -119,6 +120,7 @@ class MissileAIUpdate : public AIUpdateInterface, public ProjectileUpdateInterfa
 	Real									m_maxAccel;
 	Coord3D								m_originalTargetPos;			///< When firing uphill, we aim high to clear the brow of the hill.  jba.
 	Coord3D								m_prevPos;
+	Coord3D								m_logicStepVelocity;			///< Logic frame velocity vector
 	WeaponBonusConditionFlags		m_extraBonusFlags;
 	const WeaponTemplate*	m_detonationWeaponTmpl;		///< weapon to fire at end (or null)
 	const ParticleSystemTemplate* m_exhaustSysTmpl;

GeneralsMD/Code/GameEngine/Include/GameLogic/Module/NeutronMissileUpdate.h

@@ -103,6 +103,7 @@ class NeutronMissileUpdate : public UpdateModule,
 	virtual const Coord3D *getVelocity() const { return &m_vel; }		///< get current velocity
 	virtual void setFramesTillCountermeasureDiversionOccurs( UnsignedInt frames ) override {}
 	virtual void projectileNowJammed() override {}
+	virtual const Coord3D* getProjectileLogicVelocity() const override { return &m_logicStepVelocity; }
 
 	virtual UpdateSleepTime update() override;
 	virtual void onDelete() override;
@@ -119,6 +120,7 @@ class NeutronMissileUpdate : public UpdateModule,
 
 	Coord3D m_accel;
 	Coord3D m_vel;
+	Coord3D m_logicStepVelocity;						///< Logic frame velocity vector
 
 	UnsignedInt m_stateTimestamp;				///< time of state change
 	Bool m_isLaunched;

GeneralsMD/Code/GameEngine/Include/GameLogic/Module/UpdateModule.h

@@ -266,6 +266,7 @@ class ProjectileUpdateInterface
 	virtual Bool projectileHandleCollision(Object *other) = 0;
 	virtual void setFramesTillCountermeasureDiversionOccurs( UnsignedInt frames ) = 0; ///< Number of frames till missile diverts to countermeasures.
 	virtual void projectileNowJammed() = 0;
+	virtual const Coord3D* getProjectileLogicVelocity() const { return nullptr; }
 };
 
 //-------------------------------------------------------------------------------------------------

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

@@ -38,6 +38,7 @@
 #include "Common/DrawModule.h"
 #include "Common/FramePacer.h"
 #include "Common/GameAudio.h"
+#include "Common/GameEngine.h"
 #include "Common/GameLOD.h"
 #include "Common/GameState.h"
 #include "Common/GameUtility.h"
@@ -61,6 +62,7 @@
 #include "GameLogic/Module/StealthUpdate.h"
 #include "GameLogic/Module/StickyBombUpdate.h"
 #include "GameLogic/Module/BattlePlanUpdate.h"
+#include "GameLogic/Module/UpdateModule.h"
 #include "GameLogic/ScriptEngine.h"
 #include "GameLogic/Weapon.h"
 
@@ -412,6 +414,8 @@ Drawable::Drawable( const ThingTemplate *thingTemplate, DrawableStatusBits statu
 	//Real scaleFuzziness = thingTemplate->getInstanceScaleFuzziness();
 	//Real fuzzyScale = ( 1.0f + GameClientRandomValueReal( -scaleFuzziness, scaleFuzziness ));
 	m_instanceScale = thingTemplate->getAssetScale();// * fuzzyScale;
+	m_useExtrapolation = FALSE;
+	m_visualExtrapolationMtx.Make_Identity();
 
 	// initially not bound to an object
 	m_object = nullptr;
@@ -1149,6 +1153,18 @@ void Drawable::updateDrawable()
 	UnsignedInt now = TheGameLogic->getFrame();
 	Object *obj = getObject();
 
+	if (obj)
+	{
+		for (BehaviorModule** m = obj->getBehaviorModules(); m && *m; ++m)
+		{
+			if (ProjectileUpdateInterface* pui = (*m)->getProjectileUpdateInterface())
+			{
+				applySubFrameExtrapolation(pui);
+				break;
+			}
+		}
+	}
+
 	{
 		for (ClientUpdateModule** cu = getClientUpdateModules(); cu && *cu; ++cu)
 		{
@@ -4248,6 +4264,11 @@ void Drawable::setInstanceMatrix( const Matrix3D *instance )
 //-------------------------------------------------------------------------------------------------
 const Matrix3D *Drawable::getTransformMatrix() const
 {
+	if (m_useExtrapolation)
+	{
+		return &m_visualExtrapolationMtx;
+	}
+
 	const Object *obj = getObject();
 
 	if (obj)
@@ -5643,3 +5664,20 @@ void TintEnvelope::loadPostProcess()
 
 }
 
+
+void Drawable::applySubFrameExtrapolation(ProjectileUpdateInterface* pui)
+{
+	const Coord3D* v = pui->getProjectileLogicVelocity();
+	Real alpha = TheGameEngine->getLogicTimeAccumulator() * TheFramePacer->getActualLogicTimeScaleFps();
+
+	if (v && alpha > 0.0f && m_object)
+	{
+		m_visualExtrapolationMtx = *m_object->getTransformMatrix();
+		m_visualExtrapolationMtx.Adjust_Translation(*(Vector3*)v * (alpha > 1.0f ? 1.0f : alpha));
+		m_useExtrapolation = TRUE;
+	}
+	else
+	{
+		m_useExtrapolation = FALSE;
+	}
+}

GeneralsMD/Code/GameEngine/Source/GameLogic/Object/Behavior/DumbProjectileBehavior.cpp

@@ -121,6 +121,7 @@ DumbProjectileBehavior::DumbProjectileBehavior( Thing *thing, const ModuleData*
 	m_extraBonusFlags = 0;
 
   m_hasDetonated = FALSE;
+  m_logicStepVelocity.zero();
 }
 
 //-------------------------------------------------------------------------------------------------
@@ -579,6 +580,8 @@ void DumbProjectileBehavior::detonate()
 UpdateSleepTime DumbProjectileBehavior::update()
 {
 	const DumbProjectileBehaviorModuleData* d = getDumbProjectileBehaviorModuleData();
+	m_logicStepVelocity.zero();
+	Coord3D oldPos = *getObject()->getPosition();
 
 	if (m_lifespanFrame != 0 && TheGameLogic->getFrame() >= m_lifespanFrame)
 	{
@@ -714,6 +717,16 @@ UpdateSleepTime DumbProjectileBehavior::update()
 
 	++m_currentFlightPathStep;
 
+	if (m_currentFlightPathStep < (Int)m_flightPath.size())
+	{
+		m_logicStepVelocity = m_flightPath[m_currentFlightPathStep];
+		m_logicStepVelocity.sub( &flightStep ); // flightStep is currentPos
+	}
+	else
+	{
+		m_logicStepVelocity.zero();
+	}
+
 	return UPDATE_SLEEP_NONE;//This no longer flys with physics, so it needs to not sleep
 }
 

GeneralsMD/Code/GameEngine/Source/GameLogic/Object/Update/AIUpdate/MissileAIUpdate.cpp

@@ -136,6 +136,7 @@ MissileAIUpdate::MissileAIUpdate( Thing *thing, const ModuleData* moduleData ) :
 	m_framesTillDecoyed = 0;
 	m_noDamage = FALSE;
 	m_isJammed = FALSE;
+	m_logicStepVelocity.zero();
 }
 
 //-------------------------------------------------------------------------------------------------
@@ -731,6 +732,9 @@ UpdateSleepTime MissileAIUpdate::update()
 
 	/*UpdateSleepTime ret =*/ AIUpdateInterface::update();
 
+	m_logicStepVelocity = *getObject()->getPosition();
+	m_logicStepVelocity.sub( &newPos );
+
 #if 1
 	// srj sez: doh, why was this never in place? I guess 'cuz so few "smart" missiles ever go in
 	// a downward-towards-ground attitude. however, comanche rocket pod missiles do, so this fix

GeneralsMD/Code/GameEngine/Source/GameLogic/Object/Update/NeutronMissileUpdate.cpp

@@ -125,6 +125,7 @@ NeutronMissileUpdate::NeutronMissileUpdate( Thing *thing, const ModuleData* modu
 	m_frameAtLaunch = 0;
 
 	m_exhaustSysTmpl = nullptr;
+	m_logicStepVelocity.zero();
 
 }
 
@@ -255,13 +256,16 @@ void NeutronMissileUpdate::doLaunch()
 
 	// fall
 	Coord3D pos = *getObject()->getPosition();
+	Coord3D oldPos = pos;
 
 	pos.x += m_vel.x;
 	pos.y += m_vel.y;
 	pos.z += m_vel.z;
 
 	getObject()->setPosition( &pos );
 
+	m_logicStepVelocity = m_vel;
+
 	FXList::doFXObj(getNeutronMissileUpdateModuleData()->m_ignitionFX, getObject());
 
 	if (m_exhaustSysTmpl != nullptr)
@@ -335,6 +339,7 @@ void NeutronMissileUpdate::doAttack()
 {
 	Matrix3D mx;
 	Real speed = getNeutronMissileUpdateModuleData()->m_relativeSpeed;
+	Coord3D oldPos = *getObject()->getPosition();
 
 	if (getNeutronMissileUpdateModuleData()->m_targetFromDirectlyAbove && m_reachedIntermediatePos)
 		speed *= STRAIGHT_DOWN_SLOW_FACTOR;
@@ -423,6 +428,8 @@ void NeutronMissileUpdate::doAttack()
 	getObject()->setTransformMatrix( &mx );
 	getObject()->setPosition( &pos );
 
+	m_logicStepVelocity = m_vel;
+	m_logicStepVelocity.add( &m_accel );
 }
 
 //-------------------------------------------------------------------------------------------------
@@ -526,6 +533,12 @@ UpdateSleepTime NeutronMissileUpdate::update()
 		normal.z = -1.0f;
 		getObject()->onCollide(nullptr, getObject()->getPosition(), &normal);
 	}
+
+	if (m_state == DEAD)
+	{
+		m_logicStepVelocity.zero();
+	}
+
 	return UPDATE_SLEEP_NONE;
 }