Hi Aleksej,

Yes, you are right. They can be handled separately. Thanks!

Regards,

Michael
Sent from my iPhone

On Oct 11, 2013, at 12:20 PM, Aleksej Efimov <aleksej.efi...@oracle.com> wrote:

> Hi Michael,
> As I can see this topic was touched a little here: 
> http://mail.openjdk.java.net/pipermail/threeten-dev/2012-December/000314.html.
>  AFAIU from the above discussion the CLDR generic names were translated in 
> all locales, but the legacy JRE time zone names doesn't contain this 
> translations.  And actually we already have opened bug for this task: 
> https://bugs.openjdk.java.net/browse/JDK-8025051. I will work on it right 
> after the tzdata update and it will include this changes.
> But anyway, it's not highly related to tzdata updates. I think, this two 
> processes can go separately. Do you agree?
> 
> Thanks and Best Regards,
> Aleksej
> 
> On 11.10.2013 21:41, Michael Fang wrote:
>> Hi Aleksej,
>> 
>> I took a look at the localized TimeZoneNames_*.java files. They do not 
>> contain generic time zone names for JSR310...
>> 
>> I think we can file a separate bug to track that issue.
>> 
>> thanks,
>> 
>> -michael
>> 
>> On 13年10月10日 09:54 下午, Masayoshi Okutsu wrote:
>>> Hi Aleksej,
>>> 
>>> Here are my review comments.
>>> 
>>> - The copyright header of the data files shouldn't be removed.
>>> 
>>> - TimeZoneNames.java:
>>> 
>>> - "Middle Europe Time", "MET"}},
>>> + "MET", "MET"}},
>>> 
>>> I don't think the long name should be changed.
>>> 
>>> I didn't review the localized TimeZoneNames_*.java files. If L10N Team is 
>>> OK with them, I'm fine.
>>> 
>>> Thanks,
>>> Masayoshi
>>> 
>>> On 10/10/2013 10:30 PM, Aleksej Efimov wrote:
>>>> Hi,
>>>> 
>>>> Please, review the changes [1] needed to address the tz data update in JDK 
>>>> 8 from tzdata2013d to tzdata2013g.
>>>> 
>>>> The brief list of changes:
>>>> 1. tzdata2013g data was integrated to tzdb data files 
>>>> (make/sun/javazic/tzdata/* changes) and to sun/util/calendar test data 
>>>> files (test/sun/util/calendar/zi/tzdata/* changes).
>>>> 2. a) Updates to long time zone names
>>>> b) Updates to short name changes to address corresponding changes in 
>>>> tzdata2013e(WIT/CIT/EIT/WARST -> WIB/WITA/WIT/ART)
>>>> c) Removed unused ACT[] array
>>>> d) Added "Europe/Busingen" time zone name
>>>> All this changes a)->d) relates to 
>>>> src/share/classes/sun/util/resources/TimeZoneNames*.java files
>>>> 
>>>> The following tests were executed on JDK 8 with fix:
>>>> test/java/util/TimeZone
>>>> test/java/util/Calendar
>>>> test/java/util/Formatter
>>>> test/sun/util/calendar
>>>> test/java/time
>>>> 
>>>> Testing result: All test passed
>>>> 
>>>> Thanks!
>>>> Aleksej
>>>> 
>>>> [1] http://cr.openjdk.java.net/~aefimov/8025255/8/webrev.00/ 
>>>> <http://cr.openjdk.java.net/%7Eaefimov/8025255/8/webrev.00/>
> 

Reply via email to