Hook is weaker while hooking to left

[gerdoe-jr]

libtw2/world/src/lib.rs

Lines 737 to 748 in b928667

	pub fn saturated_add(min: f32, max: f32, value: f32, modifier: f32) -> f32 {
	if modifier < 0.0 {
	if value < min {
	return value;
	}
	} else {
	if value > max {
	return value;
	}
	}
	clamp(value + modifier, min, max)
	}

After changing this code to next given hook problem disappears though

pub fn saturated_add(min: f32, max: f32, value: f32, modifier: f32) -> f32 {
    if modifier < 0.0 {
        if value < min {
            return value;
        } else {
            return min.max(value + modifier);
        }
    } else {
        if value > max {
            return value;
        } else {
            return max.min(value + modifier);
        }
    }
}

[heinrich5991]

I don't understand what's different about the new code. Can you give example values where they differ?

[gerdoe-jr]

I have no example values, but gameplay videos, you can see the difference

weak.mp4

normal.mp4

[heinrich5991]

If you can generate these examples, perhaps you could run both functions and issue a println! statement when their result differs?

[Patiga]

I was curious about the difference between those two functions as well. I created a small playground project to get value mismatches.
https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=708e855ce0717c7690ccb6db3994cdb3

[heinrich5991]

The difference happens when the updated value is still outside the range, but closer to the accepted range:

E.g. range -1..0, value 2, modifier -1. The old code outputted 0 (a bug, I think), the new code outputs 1. Thanks for the clear reproduction example, @Patiga!

@gerdoe-jr Can you create a pull request? :)