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 09-06-2017 23:20, Naoto Sato wrote:
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.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
