Re: JDK 10 Review Request for JDK-8176583: Move currency data to /lib

2017-06-12 Thread Naoto Sato
Looks good. Naoto On 6/12/17 2:01 AM, Nishit Jain wrote: Thanks Naoto, Please check the updated webrev http://cr.openjdk.java.net/~nishjain/8176583/webrev.02/ Changes: - merged the try blocks in Currency.java - replaced the try block in CheckDataVersion.java with try-with-resource and a smal

Re: JDK 10 Review Request for JDK-8176583: Move currency data to /lib

2017-06-12 Thread Nishit Jain
Thanks Naoto, Please check the updated webrev http://cr.openjdk.java.net/~nishjain/8176583/webrev.02/ Changes: - merged the try blocks in Currency.java - replaced the try block in CheckDataVersion.java with try-with-resource and a small indentation change at line 82. Regards, Nishit Jain On 0