don't try to find the display model again in player model panel

just go with the item in that slot, the second check is expensive
and i am not sure what the need of it is.

if some greater context explains why it is needed, then this patch can be skipped.

Author: mastercoms

src/game/client/tf/vgui/tf_playermodelpanel.cpp

@@ -1349,6 +1349,9 @@ CEconItemView *CTFPlayerModelPanel::GetLoadoutItemFromMDLHandle( loadout_positio
 		if ( ( IsMiscSlot( iLoadoutSlot ) && IsMiscSlot( iPosition ) ) ||
 			 ( IsValidPickupWeaponSlot( iLoadoutSlot ) && iLoadoutSlot == iPosition ) )
 		{
+			return pItem;
+			// UNDONE: this check is making model panel absurdly expensively, and for what?
+#if 0
 			const char * pDisplayModel = pItem->GetPlayerDisplayModel( m_iCurrentClassIndex, m_iTeam );
 			if ( pDisplayModel )
 			{
@@ -1362,6 +1365,7 @@ CEconItemView *CTFPlayerModelPanel::GetLoadoutItemFromMDLHandle( loadout_positio
 				}
 				vgui::MDLCache()->Release(hMDLFindResult);
 			}
+#endif
 		}
 	}