RFR: 8151876: (tz) Support tzdata2016c

[Ramanand Patil]

Hi Roger,

I don't think this is covered in any alternate tests, because this Displayname format is newly introduced with tzdata2016b for the newly added TimeZones.

Anyway, I will double check on this to see if any test already covers this scenario otherwise I will update or add a new test case to cover this.


Regards,
Ramanand.

-----Original Message-----
From: Roger Riggs 
Sent: Monday, April 11, 2016 8:26 PM
To: core-libs-dev at openjdk.java.net
Subject: Re: RFR: 8151876: (tz) Support tzdata2016c

Hi Ramanand,

The change to the
test.java.time.format.TestZoneTextPrinterParser.test_ParseText
just eliminates the test.

Is there an alternate test that the formatter is returning the correct value for the GMT+/- cases?

Roger


On 4/11/2016 6:59 AM, Ramanand Patil wrote:
> Hi all,
>
> I would like someone from java.time to do a second review for this.
>
> Regards,
> Ramanand.
>
> -----Original Message-----
> From: Masayoshi Okutsu
> Sent: Tuesday, April 05, 2016 5:09 AM
> To: Ramanand Patil; i18n-dev at openjdk.java.net
> Cc: core-libs-dev at openjdk.java.net
> Subject: Re: RFR: 8151876: (tz) Support tzdata2016c
>
> Looks good to me. But I'd like someone from java.time to review the changes to see if it's OK for java.time.
>
> Masayoshi
>
> On 4/4/2016 6:50 PM, Ramanand Patil wrote:
>> Hi all,
>>
>> Please review the latest TZDATA (tzdata2016c) integration to JDK9.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8151876
>>
>> Webrev: http://cr.openjdk.java.net/~rpatil/8151876/webrev.00/
>>
>> All the TimeZone related tests are passed after integration.
>>
>>    
>>
>> Please note that this patch includes both tzdata2016b and tzdata2016c integration. The tzdata2016b review was abandoned because tzdata2016c was already released.
>>
>> As suggested by Masayoshi, changes are made such that,  "GMT+hh:mm" is used for formatting of the newly added TimeZones in tzdata2016b.
>>
>> [This is done to accommodate the IANA's new trial system where the 
>> new zones use numeric time zone abbreviations like "+04" instead of 
>> invented abbreviations like "ASTT".]
>>
>>    
>>
>> Regards,
>>
>> Ramanand.