Re: Review Request - 8021121: ISO 4217 Amendment Number 156

2013-08-12 Thread Yong Huang
On 2013/8/13 10:40, Yong Huang wrote: Hi Naoto, It's to add the EUR symbol in CurrencyNames_lv_LV.properties, but the old symbol LVL=Ls is not removed. LocaleDataTest is to check that EUR symbol is in CurrencyNames_lv_LV.properties. It works with the transition because the EUR symbol is in l

Re: Review Request - 8021121: ISO 4217 Amendment Number 156

2013-08-12 Thread Yong Huang
Hi Naoto, It's to add the EUR symbol in CurrencyNames_lv_LV.properties, but the old symbol LVL=Ls is not removed. LocaleDataTest is to check that EUR symbol is in CurrencyNames_lv_LV.properties. It works with the transition because the EUR symbol is in lv_LV before the transition and will not

Re: Review Request - 8021121: ISO 4217 Amendment Number 156

2013-08-12 Thread Naoto Sato
Hi Yong, Does test/sun/text/resources/LocaleDataTest work before/after the transition date? I know test/java/util/Currency tests work with the transition, but don't think the LocaleDataTest test works. Naoto On 8/12/13 1:49 AM, Yong Huang wrote: Hello All, The updated webrev is at http://c

Re: Review Request - 8021121: ISO 4217 Amendment Number 156

2013-08-12 Thread Yong Huang
Hello All, The updated webrev is at http://cr.openjdk.java.net/~yhuang/8021121/webrev.01/ http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8021121 The transition logic of currency of lv_LV is placed in source file and regression test. thanks, Yong On 2013/8/7 0:45, Naoto Sato wrote: Yo