Kajek777

Thank you for this explanation, feel free to make any changes that are better than the abomination i tried to do (i'm not too familiar with the source engine (or...

I redid the implementation according to @FlaminSarge's suggestion, using the function `IsCurrentAttackDuringDemoCharge()`, as well as added a convar `tf_shield_charge_range_extend` that disables the added check when set to 0.

Would something like this be acceptable? ```c if ( pOwner && ( pOwner->m_Shared.InCond( TF_COND_SHIELD_CHARGE ) || ( IsCurrentAttackDuringDemoCharge() && tf_shield_charge_melee_range.GetInt() ) ) ) { if (tf_shield_charge_melee_range.GetInt()) { return tf_shield_charge_melee_range.GetInt(); }...

Setting it to -1 made all attacks made during the charge impossible to hit. I could change it from `if ( IsCurrentAttackDuringDemoCharge() && tf_shield_charge_melee_range.GetInt() )` to `if ( IsCurrentAttackDuringDemoCharge() &&...

While doing some extra testing, I've noticed that the extra range is only added when I attack after shield bashing something, and not when attacking while charging, and I can't...

I was trying to do this in a way that keeps the old extender (after all, mvm samurai bots make use of it) and that can be enabled and disabled...

Seems to be working fine now; nothing changes when cvar set to a negative value or 0 (the default), range extends properly when set to positive number (tested with 128,...