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