Prevent AddMultiDamage execution with zero/negative damage

[dystopm]

Purpose

The AddMultiDamage function can be skipped entirely when no damage will be applied. While some existing cases already avoid calling it with zero damage (e.g. FireBullets3, shield hit avoids TraceAttack->AddMultiDamage calls), there are other scenarios where it is still invoked with a damage value of 0 or less (e.g., hitting a player with shotgun pellets while they have an active Shield).

These unnecessary calls not only waste processing but can also have unintended side effects for APIs implemented by developers that rely on TakeDamage being called only when damage is actually applied. This PR ensures that the function only executes when flDamage > 0.

Approach

Added a simple check at the start of AddMultiDamage to immediately return if the flDamage parameter is 0 or less.

[s1lentq]

This fix makes sense.
However, there is one detail I remember, some custom maps intentionally use negative damage values (e.g., env_laser with damage = -300) as a way to heal players.

The maps I recall include bhop_zerocheaters, deathrun_forest, deathrun_friends, but there are many more with different scenarios.

This is not a bug but rather a hidden feature that level designers sometimes rely on. With your change, such maps would stop working as intended, since TakeDamage would never be triggered.

So the fix could unintentionally break existing community content.
One possible approach could be to allow negative values (skip only flDamage == 0).

[anzz1]

I don't think flDamage == 0 should be skipped either, there is high chance that heavily customized servers might already use zero damage sources for things, to me it seems that the if damage was zero check should be done on the plugin developer's side to maintain current flexibility. It's always better the more things there are to hook into and modify than less.

[justgo97]

Sometimes plugins just need to detect the TakeDamage event regardless whether or not the damage value was bigger than zero ( Like in the example of a minigame where hitting a player does an effect or similar ), It's better to keep open and devs can easily check if flDamage <= 0.0 If that's what they need for their purposes.