JDK 9 RFR of JDK-7130085 Port fdlibm hypot to Java
Brian Burkhalter
brian.burkhalter at oracle.com
Tue Sep 22 23:20:12 UTC 2015
Hi Joe,
Overall this looks good. I only have a couple of minor observations related to internal documentation.
1. FdLibm.java
Lines 158-166: The verbiage in the note might benefit from a little reworking.
2. HypotTests.java
Line 46: “Commutative” is misspelled.
Thanks,
Brian
On Sep 21, 2015, at 6:22 PM, Joseph D. Darcy <Joe.Darcy at Oracle.Com> wrote:
> Please review the next portion of the port of fdlibm to Java:
>
> JDK-7130085 Port fdlibm hypot to Java
> http://cr.openjdk.java.net/~darcy/7130085.0/
>
> As before with pow, this isn't necessarily the end state of the code we'd like to stop at, but it should be sufficiently idiomatic Java for an initial port.
>
> To make comparison against the original C algorithm easier, I listed the C version of hypot as the base file to compare FdLibm.java against.
More information about the core-libs-dev
mailing list