Integrated: 8305505: NPE in javazic compiler

2023-05-02 Thread Yoshiki Sato
On Tue, 18 Apr 2023 05:08:35 GMT, Yoshiki Sato wrote: > Please review this PR. > With this minor change, the javazic compiler (Main.java) can produce the HTML > files that display given time zone data correctly. This pull request has now been integrated. Changeset: 544bd2

Re: RFR: 8305505: NPE in javazic compiler [v2]

2023-04-25 Thread Yoshiki Sato
> Please review this PR. > With this minor change, the javazic compiler (Main.java) can produce the HTML > files that display given time zone data correctly. Yoshiki Sato has updated the pull request incrementally with one additional commit since the last revision: Insert null

Re: RFR: 8305505: NPE in javazic compiler

2023-04-24 Thread Yoshiki Sato
On Tue, 18 Apr 2023 19:50:59 GMT, Naoto Sato wrote: >> Please review this PR. >> With this minor change, the javazic compiler (Main.java) can produce the >> HTML files that display given time zone data correctly. > > test/jdk/sun/util/calendar/zi/GenDoc.java line 159: > >> 157: if

RFR: 8305505: NPE in javazic compiler

2023-04-24 Thread Yoshiki Sato
Please review this PR. With this minor change, the javazic compiler (Main.java) can produce the HTML files that display given time zone data correctly. - Commit messages: - Remove trailing spaces - Fix a mistake in the change - 8305505: NPE in javazic compiler Changes: https://

Integrated: 8305113: (tz) Update Timezone Data to 2023c

2023-04-04 Thread Yoshiki Sato
On Fri, 31 Mar 2023 00:02:02 GMT, Yoshiki Sato wrote: > Pleases review this PR. > The PR includes the following changes. > - tzdata 2023c > - Hack code to deal with Cairo's DST end, which is same as done in > 2014g([JDK-8049343](https://bugs.openjdk.org/browse/JDK-8049343))

Re: RFR: 8305113: (tz) Update Timezone Data to 2023c [v3]

2023-04-03 Thread Yoshiki Sato
ompiler, where the time 24:00 is > recognized as 0:00 in the previous day. Yoshiki Sato has updated the pull request incrementally with two additional commits since the last revision: - Revert formatted code in ZoneInfoFile.java - fixing typo again - Changes: - all: https:

Re: RFR: 8305113: (tz) Update Timezone Data to 2023c [v2]

2023-04-03 Thread Yoshiki Sato
ompiler, where the time 24:00 is > recognized as 0:00 in the previous day. Yoshiki Sato has updated the pull request incrementally with one additional commit since the last revision: fixing some typos - Changes: - all: https://git.openjdk.org/jdk/pull/13255/files - new: h

Re: RFR: 8305113: (tz) Update Timezone Data to 2023c

2023-04-03 Thread Yoshiki Sato
On Fri, 31 Mar 2023 22:06:34 GMT, Sergey Bylokhov wrote: >> Pleases review this PR. >> The PR includes the following changes. >> - tzdata 2023c >> - Hack code to deal with Cairo's DST end, which is same as done in >> 2014g([JDK-8049343](https://bugs.openjdk.org/browse/JDK-8049343)). To work >

Re: RFR: 8305113: (tz) Update Timezone Data to 2023c

2023-04-03 Thread Yoshiki Sato
On Fri, 31 Mar 2023 22:05:01 GMT, Sergey Bylokhov wrote: >> Pleases review this PR. >> The PR includes the following changes. >> - tzdata 2023c >> - Hack code to deal with Cairo's DST end, which is same as done in >> 2014g([JDK-8049343](https://bugs.openjdk.org/browse/JDK-8049343)). To work >

Re: RFR: 8305113: (tz) Update Timezone Data to 2023c

2023-04-03 Thread Yoshiki Sato
On Fri, 31 Mar 2023 16:44:52 GMT, Naoto Sato wrote: >> Pleases review this PR. >> The PR includes the following changes. >> - tzdata 2023c >> - Hack code to deal with Cairo's DST end, which is same as done in >> 2014g([JDK-8049343](https://bugs.openjdk.org/browse/JDK-8049343)). To work >> aro

Re: RFR: 8305113: (tz) Update Timezone Data to 2023c

2023-03-30 Thread Yoshiki Sato
On Fri, 31 Mar 2023 00:02:59 GMT, Yoshiki Sato wrote: >> Pleases review this PR. >> The PR includes the following changes. >> - tzdata 2023c >> - Hack code to deal with Cairo's DST end, which is same as done in >> 2014g([JDK-8049343](https://bugs.openjdk

RFR: 8304763: (tz) Update Timezone Data to 2023a

2023-03-30 Thread Yoshiki Sato
Pleases review this PR. The PR includes the following changes. - tzdata 2023c - Hack code to deal with Cairo's DST end, which is same as done in 2014g([JDK-8049343](https://bugs.openjdk.org/browse/JDK-8049343)). To work around the the limitation of JSR 310 compiler, where the time 24:00 is rec

Re: RFR: 8304763: (tz) Update Timezone Data to 2023a

2023-03-30 Thread Yoshiki Sato
On Fri, 31 Mar 2023 00:02:02 GMT, Yoshiki Sato wrote: > Pleases review this PR. > The PR includes the following changes. > - tzdata 2023c > - Hack code to deal with Cairo's DST end, which is same as done in > 2014g([JDK-8049343](https://bugs.openjdk.org/browse/JDK-8049343))

Re: RFR: 8297804: (tz) Update Timezone Data to 2022g

2022-12-01 Thread Yoshiki Sato
On Wed, 30 Nov 2022 18:07:07 GMT, Andrew John Hughes wrote: > Update to the latest tzdata, 2022g. > > Primary changes: > * `America/Ojinaga` (CST) is split, creating `America/Ciudad_Juarez` (MST/MDT) > * `America/Pangnirtung` becomes a link to `America/Iqaluit` > * `America/Ojinaga` gains DST (C

Integrated: 8296108: (tz) Update Timezone Data to 2022f

2022-11-03 Thread Yoshiki Sato
On Wed, 2 Nov 2022 04:10:43 GMT, Yoshiki Sato wrote: > Please review this PR. The PR includes changes in the resource bundle > sources to follow the time zone change observed in America/Chihuahua and > America/Ojinaga. This pull request has now been integrated. Changeset: 9d3b4e

Re: RFR: 8296108: (tz) Update Timezone Data to 2022f

2022-11-01 Thread Yoshiki Sato
On Wed, 2 Nov 2022 04:10:43 GMT, Yoshiki Sato wrote: > Please review this PR. The PR includes changes in the resource bundle > sources to follow the time zone change observed in America/Chihuahua and > America/Ojinaga. @naotoj Can I ask you to review the PR? -

RFR: 8296108: (tz) Update Timezone Data to 2022f

2022-11-01 Thread Yoshiki Sato
Please review this PR. The PR includes changes in the resource bundle sources to follow the time zone change observed in America/Chihuahua and America/Ojinaga. - Commit messages: - 1st commit for tz2022f Changes: https://git.openjdk.org/jdk/pull/10940/files Webrev: https://webre

Re: RFR: 8295173: (tz) Update Timezone Data to 2022e

2022-10-11 Thread Yoshiki Sato
On Wed, 12 Oct 2022 00:01:50 GMT, David Alvarez wrote: > Please, review this PR for an update of timezone data. No changes besides the > import were needed. We normally run test/jdk/java/util/TimeZone/tools/share/Makefile with the JDK integrating new timezone data in order to generate test/jd

Integrated: 8294357: (tz) Update Timezone Data to 2022d

2022-09-30 Thread Yoshiki Sato
On Wed, 28 Sep 2022 03:40:06 GMT, Yoshiki Sato wrote: > Please review this PR. The change include some code changes in > ZoneInfoFile.java and TestZoneInfo310.java since tzdata2022d breaks > TestZoneInfo310.java. This pull request has now been integrated. Changeset: f01573

Re: RFR: 8294357: (tz) Update Timezone Data to 2022d

2022-09-29 Thread Yoshiki Sato
On Fri, 30 Sep 2022 04:17:54 GMT, Naoto Sato wrote: >> Please review this PR. The change include some code changes in >> ZoneInfoFile.java and TestZoneInfo310.java since tzdata2022d breaks >> TestZoneInfo310.java. > > test/jdk/java/util/TimeZone/TimeZoneData/displaynames.txt line 170: > >> 16

RFR: 8294357: (tz) Update Timezone Data to 2022d

2022-09-29 Thread Yoshiki Sato
Please review this PR. The change include some code changes in ZoneInfoFile.java and TestZoneInfo310.java since tzdata2022d breaks TestZoneInfo310.java. - Commit messages: - Added minor changes in comment lines - reverting irrelvant change in pom.xml - 1st commit for tz2022d Ch

Integrated: 8292579: (tz) Update Timezone Data to 2022c

2022-09-06 Thread Yoshiki Sato
On Thu, 25 Aug 2022 02:26:19 GMT, Yoshiki Sato wrote: > Please review this PR. The PR adopts the official tzdata2022c as it is. > It means the pre-1970s data merged in tzdata2022c doesn't exist. > All tests have been passed with no failures. This pull request has now b

Re: RFR: 8292579: (tz) Update Timezone Data to 2022c

2022-09-02 Thread Yoshiki Sato
On Thu, 25 Aug 2022 02:26:19 GMT, Yoshiki Sato wrote: > Please review this PR. The PR adopts the official tzdata2022c as it is. > It means the pre-1970s data merged in tzdata2022c doesn't exist. > All tests have been passed with no failures. The test failure seems to be irreleva

RFR: 8292579: (tz) Update Timezone Data to 2022c

2022-09-02 Thread Yoshiki Sato
Please review this PR. The PR adopts the official tzdata2022c as it is. It means the pre-1970s data merged in tzdata2022c doesn't exist. All tests have been passed with no failures. - Commit messages: - Merge branch 'master' into tz2022c - The first fix for tz2022c Changes: https:

Integrated: 8028265: Add legacy tz tests to OpenJDK

2022-07-15 Thread Yoshiki Sato
On Wed, 13 Jul 2022 04:30:39 GMT, Yoshiki Sato wrote: > Please review this PR. The PR open sources the closed timezone tests. This pull request has now been integrated. Changeset: 92deab54 Author: Yoshiki Sato Committer: Sean Coffey URL: https://git.openjdk.org/jdk/com

Re: RFR: 8028265: Add legacy tz tests to OpenJDK

2022-07-13 Thread Yoshiki Sato
On Wed, 13 Jul 2022 04:30:39 GMT, Yoshiki Sato wrote: > Please review this PR. The PR open sources the closed timezone tests. Can I ask you to help reviewing this PR? @coffeys @naotoj - PR: https://git.openjdk.org/jdk/pull/9476

RFR: 8028265: Add legacy tz tests to OpenJDK

2022-07-13 Thread Yoshiki Sato
Please review this PR. The PR open sources the closed timezone tests. - Commit messages: - Minor change added to Makefile - 8028265: Move closed timezone tests to open repository Changes: https://git.openjdk.org/jdk/pull/9476/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr