On 2013/8/14 0:24, Naoto Sato wrote:
On 8/12/13 7:40 PM, 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 lv_LV before the transition and
will not be removed after the transition.

You meant "Ls" here, right?

Yes, both EUR symbol and "Ls" will remain in the resource file after the transition.

thanks for the review,
Yong


LVL=Ls will remain in the file CurrencyNames_lv_LV.properties after the
transition but will not bring any regression.

OK, looks good to me.

Naoto


thanks,
Yong


On 2013/8/13 6:03, Naoto Sato wrote:
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://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:
Yong,

There already is a logic in Currency class to deal with such a future
transition nicely. That's what I meant.

Masayoshi,

Then it will be a regression till 1/1/2014. I don't see any problem
using the transition logic here, IMO.

Naoto

On 8/6/13 2:13 AM, Masayoshi Okutsu wrote:
Hi Yong,

I think it's OK to use EUR without specifying the transition date-time
for JDK 8.

Thanks,
Masayoshi

On 8/6/2013 11:44 AM, Yong Huang wrote:
Hi Naoto,

To place the transition logic, do you mean I need to add comments in
source file or do I just need to explain it in this email thread?

The currency change will take place in January 2014 and Java 8 is
still not released at that time. So when Java 8 is released, the
currency of LV will already be changed. If I fix the bug after Jan
2014, it will be near the release date of Java 8 and maybe only
stopper bug is allowed to fixed at that time.

thanks,
Yong

On 2013/8/6 2:32, Naoto Sato wrote:
At the moment, they still use LVL for their currency, so I think the
transition logic needs to be placed.

Naoto

On 8/4/13 7:46 PM, Yong Huang wrote:
Hello,

This is the review request for
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8021121

Webrev: http://cr.openjdk.java.net/~yhuang/8021121/webrev.00/

thanks,
Yong









Reply via email to