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

2017-06-09 Thread Naoto Sato
Hi Nishit, In Currency.java, there seems to be a redundant try block (line 206). Is it possible to merge it with the try block at line 205? Otherwise it looks good to me. Naoto On 6/8/17 11:59 PM, Nishit Jain wrote: Hi, Please review the fix for JDK-8176583 Bug: https://bugs.openjdk.java.

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

2017-06-09 Thread Nishit Jain
Hi, Please review the fix for JDK-8176583 Bug: https://bugs.openjdk.java.net/browse/JDK-8176583 Webrev: http://cr.openjdk.java.net/~nishjain/8176583/webrev.01/ Fix: Relocated currency.data from java.base module (/java.base/java/util/currency.data) to /lib directory Regards, Nishit Jain