RFR: 8301226: Add clamp() methods to java.lang.Math and to StrictMath [v2]

ExE Boss duke at openjdk.org
Sun Feb 5 18:21:57 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

That should probably include a comment then.

-------------

PR: https://git.openjdk.org/jdk/pull/12428


More information about the core-libs-dev mailing list