Hi,
Please review below fix :
Bug: https://bugs.openjdk.java.net/browse/JDK-8202696
Webrev: http://cr.openjdk.java.net/~dkumar/8202696/webrev.00/
Characters which are not getting displayed actually belong to Phonetic
Extensions (https://en.wikipedia.org/wiki/Phonetic_Extensions ).
These chara
Hi Naoto,
I removed the obsoleted currency LTL and LVL from tablea1.txt, added
them into otherCodes in ValidateISO4217.java.
new webrev as below:
http://cr.openjdk.java.net/~ljiang/8202026/webrev.03/
Thanks,
Leo
On 05/06/2018 2:27 AM, naoto.s...@oracle.com wrote:
Hi Leo,
Change looks good.
Hi,
Please review the fix for JDK-8203872.
Bug: https://bugs.openjdk.java.net/browse/JDK-8203872
Webrev: http://cr.openjdk.java.net/~nishjain/8203872/webrev.01/
Fix: Updated the LSR data to the version 2017-08-15
Regards,
Nishit Jain
Hi Leo,
Change looks good. One leftover from the previous:
>> in CurrencyData.properties. This applies to tabela1.txt as well.
Can you please clean those LV/LT entries in tablea1.txt as well?
Naoto
On 6/4/18 7:41 AM, li.ji...@oracle.com wrote:
Hi Naoto,
Pls review the updated code:
http://
Hi Naoto,
Pls review the updated code:
http://cr.openjdk.java.net/~ljiang/8202026/webrev.02/
- As suggested, clean up the old transition dates.
- Update copyright year.
- ISO 4217 Amendment #167 was published today, which discards the #166,
so withdraw the change for #166 in webrev.02.
Bug f
