<i18n dev> Review Request - 8021121: ISO 4217 Amendment Number 156

[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://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
>>>>>
>>>>
>>>
>>
>