PING! - RFR 7032154: Performance tuning of sun.misc.FloatingDecimal/FormattedFloatingDecimal
Brian Burkhalter
brian.burkhalter at oracle.com
Thu May 30 21:18:32 UTC 2013
Hi Martin,
The changes you suggested below have been made and the webrev updated at
http://cr.openjdk.java.net/~bpb/7032154/
All pertinent JTREG and JCK tests have been re-run successfully.
Thanks!
Brian
On May 29, 2013, at 4:57 PM, Martin Buchholz wrote:
> The code is awesome. My thorough review found only these defects:
>
> Use third person in the first sentence of a javadoc - e.g. s/Retrieve/Retrieves/.
>
> ---
>
> Don't use the denigrated C style
> char zero[] =
> Instead use
> char[] zero =
>
> ---
> This looks odd:
> normalized as a//binary*
>
> The original comment looked more normal:
>
> normalized as a *binary*
> ---
>
> Otherwise, approved!
More information about the core-libs-dev
mailing list