Re: RFR: 8210153 : java/text/Format/NumberFormat/CurrencyFormat.java failed

2018-08-29 Thread naoto . sato
Please update the copyright year to "2018". Otherwise looks good to me. No further review is needed. Naoto On 8/29/18 4:41 PM, li.ji...@oracle.com wrote: Hi Naoto, Please review the fix to update the currency symbol for VES. Bug: https://bugs.openjdk.java.net/browse/JDK-8210153 Webrev: htt

Re: RFR: 8208746 8209775 : ISO 4217 Amendment #168 #169 Update

2018-08-29 Thread Joseph D. Darcy
Hi Naoto, Thanks for checking; cheers, -Joe On 8/28/2018 6:44 PM, naoto.s...@oracle.com wrote: Hi Joe, This is a typical currency data change, and does not involve any public interface changes, so I think CSR is not needed. We haven't filed CSRs for such changes in the past. Naoto On 8/2

RFR: 8210153 : java/text/Format/NumberFormat/CurrencyFormat.java failed

2018-08-29 Thread li . jiang
Hi Naoto, Please review the fix to update the currency symbol for VES. Bug: https://bugs.openjdk.java.net/browse/JDK-8210153 Webrev: http://cr.openjdk.java.net/~ljiang/8210153/webrev/ Built and tier1~tier3 tests passed on mach5. Thanks, Leo