<i18n dev> Review Request-JDK-8154797:Localization data for "GMT"

Masayoshi Okutsu masayoshi.okutsu at oracle.com
Wed Jul 13 23:25:47 UTC 2016


+1

Masayoshi


On 7/14/2016 2:15 AM, Naoto Sato wrote:
> +1
>
> Naoto
>
> On 7/13/16 10:14 AM, Rachna Goel wrote:
>> Hi Naoto,
>>
>> Thanks for the review.
>> Please have a look at updated webrev :
>> http://cr.openjdk.java.net/~rgoel/JDK-8154797/webrev.05/
>>
>> Thanks,
>> Rachna
>>
>> On 7/13/2016 9:04 PM, Naoto Sato wrote:
>>> Hi Rachna,
>>>
>>> Did you intend to use Locale.GERMANY/JAPAN/US, instead of
>>> Locale.GERMAN/JAPANESE/ENGLISH in Bug8154797.java? You can also use
>>> Locale.FRANCE and Locale.UK constants in the test case.
>>>
>>> Otherwise it looks good.
>>>
>>> Naoto
>>>
>>> On 7/13/16 2:01 AM, Rachna Goel wrote:
>>>> Hi,
>>>>
>>>> Please review fix for JDK-8154797.
>>>>
>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8154797
>>>>
>>>> webrev: http://cr.openjdk.java.net/~rgoel/JDK-8154797/webrev.04/
>>>>
>>>> fix summary: Added support for GMT Localization resources 
>>>> (GmtFormat and
>>>> hourFormat).
>>>> For more info:
>>>> http://unicode.org/repos/cldr/trunk/specs/ldml/tr35-dates.html#Using_Time_Zone_Names 
>>>>
>>>>
>>>>
>>>>
>>



More information about the i18n-dev mailing list