Update Recoil / Spread to be flat reduction, capping out at 100%, allow knockback to go above 100%

[secretagentjr]

No Recoil/Spread at 100% for each stat, Knockback can go above 100%, Health displays properly past 255

[GenZmeY]

It doesn't look right. type of PlayerHealthPercent is a byte and it can have values in the range 0-255. Since percentages are stored there (range 0-100), I don't see any problems with the old code.

Speaking of new code:

PlayerHealthPercent *= float(OwnerPawn.Health) / float(OwnerPawn.HealthMax);

The float(OwnerPawn.Health) / float(OwnerPawn.HealthMax); will always give values in the range 0.0f - 1.0f, and multiplying this result by the previous value of PlayerHealthPercent ( *= ) will decrement PlayerHealthPercent each time it is updated

If health is incorrectly displayed somewhere in the interface, then most likely the problem is something else. Can you tell me how to reproduce the bug you are trying to fix here?

Also added some logs and found that this function is useless: we never get inside the if because both variables always have the actual value:

It looks like they are updated somewhere else.

In this case (+ what I said earlier), I see no reason to change this function.