RFR 7032154: Performance tuning of sun.misc.FloatingDecimal/FormattedFloatingDecimal (original) (raw)
Brian Burkhalter brian.burkhalter at oracle.com
Thu May 30 21🔞32 UTC 2013
- Previous message: PING! - RFR 7032154: Performance tuning of sun.misc.FloatingDecimal/FormattedFloatingDecimal
- Next message: PING! - RFR 7032154: Performance tuning of sun.misc.FloatingDecimal/FormattedFloatingDecimal
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
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!
- Previous message: PING! - RFR 7032154: Performance tuning of sun.misc.FloatingDecimal/FormattedFloatingDecimal
- Next message: PING! - RFR 7032154: Performance tuning of sun.misc.FloatingDecimal/FormattedFloatingDecimal
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]