<i18n dev> Review Request for JDK-8150324: java/util/Currency/CurrencyTest.java does not restore default TimeZone

Masayoshi Okutsu masayoshi.okutsu at oracle.com
Mon Apr 11 13:35:29 UTC 2016


Looks good to me.

Masayoshi

On 4/11/2016 9:18 PM, Nishit Jain wrote:
> Hi,
>
> Thanks for pointing that out.
>
> Please review the updated webrev at the below mentioned link
>
> http://cr.openjdk.java.net/~peytoia/Nishit/8150324/webrev.02/
>
> Regards,
> Nishit Jain
>
> On 4/11/2016 4:46 PM, Yuka Kamiya wrote:
>> Hi Nishit,
>>
>> Can you please replace 
>> "jdk/src/share/classes/java/util/CurrencyData.properties" in a 
>> comment in the test program with 
>> "jdk/make/data/currency/CurrencyData.properties" ?
>>
>> Otherwise, your fix looks good to me.
>>
>> Thanks,
>> -- 
>> Yuka
>>
>> On 2016/04/11 19:55, Nishit Jain wrote:
>>> Hello All,
>>>
>>> Please review the following fix for JDK-8150324
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8150324
>>> Webrev: http://cr.openjdk.java.net/~peytoia/Nishit/8150324/webrev.01/
>>>
>>> Fix: Tried to resolve the issue using java.time APIs to avoid 
>>> problems caused by setting the default TimeZone of the system.
>>>
>>> Regards,
>>> Nishit Jain
>>
>



More information about the i18n-dev mailing list