RFR: 8301226: Add clamp() methods to java.lang.Math and to StrictMath [v2]
Quan Anh Mai
qamai at openjdk.org
Mon Feb 6 03:28:50 UTC 2023
On Sun, 5 Feb 2023 13:52:23 GMT, Tagir F. Valeev <tvaleev at openjdk.org> wrote:
>> src/java.base/share/classes/java/lang/Math.java line 2297:
>>
>>> 2295: */
>>> 2296: public static float clamp(float value, float min, float max) {
>>> 2297: if (!(min < max)) {
>>
>> Wouldn't it be more simple to replace `!(min < max)` with `min >= max`?
>
> No. I want NaNs to go into this branch
@amaembo Should that be `if (!(min <= max))` instead?
-------------
PR: https://git.openjdk.org/jdk/pull/12428
More information about the core-libs-dev
mailing list