Vec2 clamp_length alters vector when length is within specified range about glam-rs HOT 5 CLOSED

Looking into this some more, I think part of the issue is negative values probably shouldn't be allowed with this function. Basically vector length cannot be less than zero, the clamp function makes sure then length is no less than the given min, but the doesn't really make sense if min (or max for that matter are negative. What should probably happen here is that the docs should say that min and max must be positive and we could add some glam_assets to check the inputs.

from glam-rs.

Thanks for the report, that definitely sounds wrong. I think I can see what the problem is in the implementation, I'll look at fixing that soon.

from glam-rs.

I've sent a draft PR for the f32 Vec2 method. If it looks good, I can update the other methods afterward.

from glam-rs.

I agree. It's much better

from glam-rs.