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.