Well, you are also being stubborn. Plus, I see nothing rude from him (though what you quoted was unnecessary), he's actually helping you... Maybe you should calm down a bit, and trying to review how work things.
Anyway, what he said should be correct. You may not see at first glance any problems, but at some point you are not going to have reliable damage. By calling TraceAttack, you accumulate damage for a same entity. If you call TakeDamage wihtout clearing global multi-damage, once TraceAttack is called again but with another entity, The previous entity will take most likely and again the accumulated damage with different inflictor.
This is not for nothing you have such code for the knife for example:
Code:
ClearMultiDamage();
pEntity->TraceAttack(m_pPlayer->pev, flDamage, gpGlobals->v_forward, &tr, DMG_NEVERGIB | DMG_BULLET);
ApplyMultiDamage(m_pPlayer->pev, m_pPlayer->pev);
__________________