Rémi Forax wrote:
On 12/01/2010 06:24 PM, Joe Darcy wrote:
Hello.
Please review my simple fix for
7002594 "Math.max and Math.min should use floatToRawIntBits() to
check for -0.0"
http://cr.openjdk.java.net/~darcy/7002594.0/
Besides addressing the reported issues with max/min, I also review
the other uses of those methods in java.lang.* and added a comment or
two. While looking over sun/misc/FpUtils.java, I took the
opportunity to update the javadoc style.
Thanks,
-Joe
Looks fine.
I just wonder why twoToTheDoubleScaleUp and twoToTheDoubleScaleDown in
FpUtils
aren't real constants (static final).
Just an oversight; they are logically constants -- the next time I edit
this class, I'll fix that.
Thanks,
-Joe