Looks good to me.

Naoto

On 2/18/16 2:08 AM, Nishit Jain wrote:
Hello All,

Please review the updated fix in "/Currency.java/".

The updated fix is located at:

Webrev: http://cr.openjdk.java.net/~rgoel/Nishit/8074411/webrev.01/
<http://cr.openjdk.java.net/%7Ergoel/Nishit/8074411/webrev.01/>

Changes made in the current version (webrev.01): Rephrasing of text in
the /getDefaultFractionDigits()/ method description.

Regards,
Nishit Jain

On 2/15/2016 5:38 PM, Nishit Jain wrote:
Hello All,

Please review the following fix in "/Currency.java/".

Bug: https://bugs.openjdk.java.net/browse/JDK-8074411
Webrev: http://cr.openjdk.java.net/~rgoel/Nishit/8074411/
<http://cr.openjdk.java.net/%7Ergoel/Nishit/8074411/>

Fix: A clarification about "minor units" and "default number of
fraction digits" is added in the specification. A suggestion to use
BigDecimal class while working with Currency or monetary values is
also added.

Regards,
Nishit Jain

Reply via email to