lowered weapon shooting fix

game/shared/basecombatweapon_shared.cpp

@@ -1647,30 +1647,34 @@ void CBaseCombatWeapon::ItemPreFrame( void )
 //====================================================================================
 // WEAPON BEHAVIOUR
 //====================================================================================
-void CBaseCombatWeapon::ItemPostFrame( void )
+void CBaseCombatWeapon::ItemPostFrame(void)
 {
-	CBasePlayer *pOwner = ToBasePlayer( GetOwner() );
+	CBasePlayer* pOwner = ToBasePlayer(GetOwner());
 	if (!pOwner)
 		return;
 
+
 	UpdateAutoFire();
 
 	//Track the duration of the fire
 	//FIXME: Check for IN_ATTACK2 as well?
 	//FIXME: What if we're calling ItemBusyFrame?
-	m_fFireDuration = ( pOwner->m_nButtons & IN_ATTACK ) ? ( m_fFireDuration + gpGlobals->frametime ) : 0.0f;
+	m_fFireDuration = (pOwner->m_nButtons & IN_ATTACK) ? (m_fFireDuration + gpGlobals->frametime) : 0.0f;
 
-	if ( UsesClipsForAmmo1() )
+	if (UsesClipsForAmmo1())
 	{
 		CheckReload();
 	}
 
 	bool bFired = false;
-
+#ifdef GAME_DLL
+	if (!m_bLowered)
+	{
+#endif
 		// Secondary attack has priority
 		if ((pOwner->m_nButtons & IN_ATTACK2) && (m_flNextSecondaryAttack <= gpGlobals->curtime))
 		{
-		if (UsesSecondaryAmmo() && pOwner->GetAmmoCount(m_iSecondaryAmmoType)<=0 )
+			if (UsesSecondaryAmmo() && pOwner->GetAmmoCount(m_iSecondaryAmmoType) <= 0)
 			{
 				if (m_flNextEmptySoundTime < gpGlobals->curtime)
 				{
@@ -1692,7 +1696,7 @@ void CBaseCombatWeapon::ItemPostFrame( void )
 				// stops the crossbow from firing on the 360 if the player chooses to hold down their
 				// zoom button. (sjb) Orange Box 7/25/2007
 #if !defined(CLIENT_DLL)
-			if( !IsX360() || !ClassMatches("weapon_crossbow") )
+				if (!IsX360() || !ClassMatches("weapon_crossbow"))
 #endif
 				{
 					bFired = ShouldBlockPrimaryFire();
@@ -1701,23 +1705,23 @@ void CBaseCombatWeapon::ItemPostFrame( void )
 				SecondaryAttack();
 
 				// Secondary ammo doesn't have a reload animation
-			if ( UsesClipsForAmmo2() )
+				if (UsesClipsForAmmo2())
 				{
 					// reload clip2 if empty
 					if (m_iClip2 < 1)
 					{
-					pOwner->RemoveAmmo( 1, m_iSecondaryAmmoType );
+						pOwner->RemoveAmmo(1, m_iSecondaryAmmoType);
 						m_iClip2 = m_iClip2 + 1;
 					}
 				}
 			}
 		}
 
-	if ( !bFired && (pOwner->m_nButtons & IN_ATTACK) && (m_flNextPrimaryAttack <= gpGlobals->curtime))
+		if (!bFired && (pOwner->m_nButtons & IN_ATTACK) && (m_flNextPrimaryAttack <= gpGlobals->curtime))
 		{
 			// Clip empty? Or out of ammo on a no-clip weapon?
-		if ( !IsMeleeWeapon() &&  
+			if (!IsMeleeWeapon() &&
-			(( UsesClipsForAmmo1() && m_iClip1 <= 0) || ( !UsesClipsForAmmo1() && pOwner->GetAmmoCount(m_iPrimaryAmmoType)<=0 )) )
+				((UsesClipsForAmmo1() && m_iClip1 <= 0) || (!UsesClipsForAmmo1() && pOwner->GetAmmoCount(m_iPrimaryAmmoType) <= 0)))
 			{
 				HandleFireOnEmpty();
 			}
@@ -1730,6 +1734,7 @@ void CBaseCombatWeapon::ItemPostFrame( void )
 			}
 			else
 			{
+
 				//NOTENOTE: There is a bug with this code with regards to the way machine guns catch the leading edge trigger
 				//			on the player hitting the attack key.  It relies on the gun catching that case in the same frame.
 				//			However, because the player can also be doing a secondary attack, the edge trigger may be missed.
@@ -1737,28 +1742,32 @@ void CBaseCombatWeapon::ItemPostFrame( void )
 				//			first shot.  Right now that's too much of an architecture change -- jdw
 
 				// If the firing button was just pressed, or the alt-fire just released, reset the firing time
-			if ( ( pOwner->m_afButtonPressed & IN_ATTACK ) || ( pOwner->m_afButtonReleased & IN_ATTACK2 ) )
+				if ((pOwner->m_afButtonPressed & IN_ATTACK) || (pOwner->m_afButtonReleased & IN_ATTACK2))
 				{
 					m_flNextPrimaryAttack = gpGlobals->curtime;
 				}
 
 				PrimaryAttack();
 
-			if ( AutoFiresFullClip() )
+				if (AutoFiresFullClip())
 				{
 					m_bFiringWholeClip = true;
 				}
 
 #ifdef CLIENT_DLL
-			pOwner->SetFiredWeapon( true );
+				pOwner->SetFiredWeapon(true);
 #endif
 			}
 		}
+#ifdef GAME_DLL
+	}
+#endif
+
 
 	// -----------------------
 	//  Reload pressed / Clip Empty
 	// -----------------------
-	if ( ( pOwner->m_nButtons & IN_RELOAD ) && UsesClipsForAmmo1() && !m_bInReload ) 
+	if ((pOwner->m_nButtons & IN_RELOAD) && UsesClipsForAmmo1() && !m_bInReload)
 	{
 		// reload when reload is pressed, or if no buttons are down and weapon is empty.
 		Reload();
@@ -1771,7 +1780,7 @@ void CBaseCombatWeapon::ItemPostFrame( void )
 	if (!((pOwner->m_nButtons & IN_ATTACK) || (pOwner->m_nButtons & IN_ATTACK2) || (CanReload() && pOwner->m_nButtons & IN_RELOAD)))
 	{
 		// no fire buttons down or reloading
-		if ( !ReloadOrSwitchWeapons() && ( m_bInReload == false ) )
+		if (!ReloadOrSwitchWeapons() && (m_bInReload == false))
 		{
 			WeaponIdle();
 		}

game/shared/basecombatweapon_shared.h

@@ -526,6 +526,12 @@ private:
 	CNetworkVar( CBaseCombatCharacterHandle, m_hOwner );				// Player carrying this weapon
 
 protected:
+
+#ifdef GAME_DLL
+	bool			m_bLowered;			// Whether the viewmodel is raised or lowered
+	float			m_flRaiseTime;		// If lowered, the time we should raise the viewmodel
+#endif
+
 #if defined ( TF_CLIENT_DLL ) || defined ( TF_DLL )
 	// Regulate crit frequency to reduce client-side seed hacking
 	void					AddToCritBucket( float flAmount );

game/shared/hl2/basehlcombatweapon_shared.h

@@ -58,8 +58,6 @@ public:
 
 protected:
 
-	bool			m_bLowered;			// Whether the viewmodel is raised or lowered
-	float			m_flRaiseTime;		// If lowered, the time we should raise the viewmodel
 	float			m_flHolsterTime;	// When the weapon was holstered
 };