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-...@openjdk.java.net
Cc: core-libs-dev@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.

Reply via email to