Looks good.

Naoto

On 11/7/17 3:41 AM, Ramanand Patil wrote:
Hi Naoto,
Thank you for pointing that. I have modified both the affected 
files(ZoneName.java from src and test).
Here is the updated Webrev: 
http://cr.openjdk.java.net/~rpatil/8190258+8190259/webrev.01/

Regards,
Ramanand.

-----Original Message-----
From: Naoto Sato
Sent: Tuesday, November 07, 2017 5:18 AM
To: Ramanand Patil <ramanand.pa...@oracle.com>; core-libs-dev <core-libs-
d...@openjdk.java.net>; i18n-...@openjdk.java.net
Subject: Re: <i18n dev> JDK10 RFR of JDK-8190258: (tz) Support tzdata2017c
and 8190259: test tck.java.time.zone.TCKZoneRules is broken by tzdata2017c

Hi Ramanand,

In java/time/format/ZoneName.java, should the zid for Africa_Central map
to "Africa/Maputo"?

Naoto

On 11/6/17 6:06 AM, Ramanand Patil wrote:
Hi all,
Please review the latest TZDATA integration (tzdata2017c) to JDK10.
Bugs:
https://bugs.openjdk.java.net/browse/JDK-8190258
https://bugs.openjdk.java.net/browse/JDK-8190259

Webrev: http://cr.openjdk.java.net/~rpatil/8190258+8190259/webrev.00/

All the TimeZone related tests are passed after integration.

Regards,
Ramanand.

Reply via email to