RFR: 8286271: [java.desktop] Replace ternary operators with java.lang.Math min/max
Kevin Rushforth
kcr at openjdk.java.net
Fri May 6 21:25:48 UTC 2022
On Sat, 15 Jan 2022 04:23:33 GMT, SWinxy <duke at openjdk.java.net> wrote:
> Delete `sun.java2d.marlin.FloatMath` class in favor of methods in `java.lang.Math`. The methods in `Math.java` are intrinsics and can gain from significant speedups. Hand-written min/max operations are also replaced with calls. 5 uses of `ceil` and 2 uses of `floor` were replaced.
> This should not cause any rendering differences from the different algorithms being used, as they theoretically will end up at the same outcomes.
I agree with Phil. Whatever small benefit there might be in doing this isn't justified by the cost of reviewing and testing it, especially given the very real risk of introducing regressions.
This PR should be withdrawn and the JBS bug closed as "Won't fix"
-------------
Changes requested by kcr (Author).
PR: https://git.openjdk.java.net/jdk/pull/7097
More information about the client-libs-dev
mailing list