Replace usage of m_ForceSendAnimations with entvars to avoid changing the class data layout (#306)

dlls/player.cpp

@@ -3095,15 +3095,9 @@ void CBasePlayer::SelectItem( const char *pstr )
 
 	if( m_pActiveItem )
 	{
-		CBasePlayerWeapon* weapon = (CBasePlayerWeapon*)(m_pActiveItem->GetWeaponPtr());
+		m_pActiveItem->pev->oldbuttons = 1;
-		if (weapon)
-			weapon->m_ForceSendAnimations = true;
-
 		m_pActiveItem->Deploy();
-
+		m_pActiveItem->pev->oldbuttons = 0;
-		if (weapon)
-			weapon->m_ForceSendAnimations = false;
-
 		m_pActiveItem->UpdateItemInfo();
 	}
 }
@@ -3130,14 +3124,9 @@ void CBasePlayer::SelectLastItem( void )
 	m_pActiveItem = m_pLastItem;
 	m_pLastItem = pTemp;
 
-	CBasePlayerWeapon* weapon = (CBasePlayerWeapon*)(m_pActiveItem->GetWeaponPtr());
+	m_pActiveItem->pev->oldbuttons = 1;
-	if (weapon)
-		weapon->m_ForceSendAnimations = true;
-
 	m_pActiveItem->Deploy();
-
+	m_pActiveItem->pev->oldbuttons = 0;
-	if (weapon)
-		weapon->m_ForceSendAnimations = false;
 
 	m_pActiveItem->UpdateItemInfo();
 }
@@ -4610,14 +4599,9 @@ BOOL CBasePlayer::SwitchWeapon( CBasePlayerItem *pWeapon )
 
 	m_pActiveItem = pWeapon;
 
-	CBasePlayerWeapon* weapon = (CBasePlayerWeapon*)(pWeapon->GetWeaponPtr());
+	pWeapon->pev->oldbuttons = 1;
-	if (weapon)
-		weapon->m_ForceSendAnimations = true;
-
 	pWeapon->Deploy();
-
+	pWeapon->pev->oldbuttons = 0;
-	if (weapon)
-		weapon->m_ForceSendAnimations = false;
 
 	return TRUE;
 }

dlls/weapons.cpp

@@ -820,7 +820,7 @@ int CBasePlayerWeapon::UpdateClientData( CBasePlayer *pPlayer )
 void CBasePlayerWeapon::SendWeaponAnim( int iAnim, int skiplocal, int body )
 {
 	if( UseDecrement() )
-		skiplocal = !m_ForceSendAnimations;
+		skiplocal = !pev->oldbuttons;
 	else
 		skiplocal = 0;
 

dlls/weapons.h

@@ -349,9 +349,6 @@ public:
 	// hle time creep vars
 	float	m_flPrevPrimaryAttack;
 	float	m_flLastFireTime;
-
-	//Hack so deploy animations work when weapon prediction is enabled.
-	bool m_ForceSendAnimations;
 };
 
 class CBasePlayerAmmo : public CBaseEntity