networking: use >= when checking for full updates!

Author: RaphaelIT7

source/modules/networking.cpp

@@ -1285,15 +1285,17 @@ struct PlayerTransmitCache
 		if (nTransmitTicks == -1) {
 			CBaseClient* pClient = Util::GetClientByPlayer((const CBasePlayer*)pPlayer);
 			if (pClient) {
-				nLastAcknowledgedTick = pClient->GetMaxAckTickCount();
+				nLastAcknowledgedTick = pClient->GetMaxAckTickCount(); // pClient->m_nDeltaTick;
+				// Verify: GetMaxAckTickCount may be inaccurate for our use case since we need m_nDeltaTick?
+				// if (pClient->m_nDeltaTick != pClient->GetMaxAckTickCount())
+				// 	DevMsg(PROJECT_NAME " - networking: Interresting... for client %i (ent index) the delta tick %i differs from the MaxAckTick %i (%i)\n", pPlayer->edict()->m_EdictIndex, pClient->m_nDeltaTick, pClient->GetMaxAckTickCount(), nFullUpdateTick);
 			} else {
-				DevMsg(PROJECT_NAME " - networking: Failed to get CBaseClient for player %i (ent index)\n", pPlayer->edict()->m_EdictIndex);
+				// DevMsg(PROJECT_NAME " - networking: Failed to get CBaseClient for player %i (ent index)\n", pPlayer->edict()->m_EdictIndex);
 				nLastAcknowledgedTick = nTick - 100; // Fallback though should never happen
 			}
 		} else {
 			nLastAcknowledgedTick = nTick - nTransmitTicks;
 		}
-
 
 		for (int i=0; i<MAX_WEAPONS; ++i)
 		{
@@ -1330,6 +1332,7 @@ struct PlayerTransmitCache
 
 	void MarkFullUpdate()
 	{
+		// DevMsg(PROJECT_NAME " - networking: Triggered fullupdate %i\n", gpGlobals->tickcount);
 		nFullUpdateTick = gpGlobals->tickcount;
 	}
 
@@ -1341,7 +1344,8 @@ struct PlayerTransmitCache
 	// If the client relative to his own last acknowledged tick
 	bool InFullUpdate() const
 	{
-		return nFullUpdateTick > nLastAcknowledgedTick;
+		// DevMsg(PROJECT_NAME " - networking: InFullUpdate %i - %i\n", nFullUpdateTick, nLastAcknowledgedTick);
+		return nFullUpdateTick >= nLastAcknowledgedTick;
 	}
 
 	bool bIsValid = false;
@@ -1575,6 +1579,7 @@ static void hook_CBaseCombatCharacter_SetTransmit(CBaseCombatCharacter* pCharact
 			const PlayerTransmitCache& pCache = g_pPlayerTransmitCache[pCharacterEdict->m_EdictIndex-1];
 			if (networking_transmit_onfullupdate.GetBool() && pCache.InFullUpdate())
 			{
+				// DevMsg(PROJECT_NAME " - networking: Doing full weapon transmit for %i...\n", pCharacterEdict->m_EdictIndex);
 				for (int i=0; i < MAX_WEAPONS; ++i)
 				{
 					CBaseEntity *pWeapon = GetMyWeapon(pCharacter, i);