game\client&server: Invalidate bone cache same way in server and client + prevent future bone cache times

See ValveSoftware/source-sdk-2013#891

Author: dimhotepus

game/client/c_baseanimating.cpp

@@ -2332,7 +2332,9 @@ CBoneCache *C_BaseAnimating::GetBoneCache( CStudioHdr *pStudioHdr )
 	CBoneCache *pcache = Studio_GetBoneCache( m_hitboxBoneCacheHandle );
 	if ( pcache )
 	{
-		if ( pcache->IsValid( gpGlobals->curtime, 0.0 ) )
+		// dimhotepus: Use default 0.1 jitter as server does and ensure (as server does) bone cache
+		// is not from future (due to prediction).
+		if ( pcache->IsValid( gpGlobals->curtime ) && pcache->m_timeValid <= gpGlobals->curtime )
 		{
 			// in memory and still valid, use it!
 			return pcache;

game/server/baseanimating.cpp

@@ -455,7 +455,8 @@ void CBaseAnimating::StudioFrameAdvanceInternal( CStudioHdr *pStudioHdr, float f
 void CBaseAnimating::InvalidateBoneCacheIfOlderThan( float deltaTime )
 {
 	CBoneCache *pcache = Studio_GetBoneCache( m_boneCacheHandle );
-	if ( !pcache || !pcache->IsValid( gpGlobals->curtime, deltaTime ) )
+	// dimhotepus: Invalidate bone cache if its old or in the future.
+	if ( !pcache || !pcache->IsValid( gpGlobals->curtime, deltaTime ) || pcache->m_timeValid > gpGlobals->curtime )
 	{
 		InvalidateBoneCache();
 	}
@@ -2623,6 +2624,7 @@ CBoneCache *CBaseAnimating::GetBoneCache( void )
 #endif
 	if ( pcache )
 	{
+		// dimhotepus: If you change this, sync with C_BaseAnimating::GetBoneCache
 		if ( pcache->IsValid( gpGlobals->curtime ) && (pcache->m_boneMask & boneMask) == boneMask && pcache->m_timeValid <= gpGlobals->curtime)
 		{
 			// Msg("%s:%s:%s (%x:%x:%8.4f) cache\n", GetClassname(), GetDebugName(), STRING(GetModelName()), boneMask, pcache->m_boneMask, pcache->m_timeValid );