Re: RFR: 8282131: java.time.ZoneId should be a sealed abstract class

2022-02-25 Thread Stephen Colebourne 
On Fri, 25 Feb 2022 19:02:55 GMT, Naoto Sato  wrote:

> Refactoring `java.time.ZoneId` class to be a sealed class. A CSR has also 
> been drafted.

Marked as reviewed by scolebourne (Author).

-

PR: https://git.openjdk.java.net/jdk/pull/7625

Re: RFR: 8282131: java.time.ZoneId should be a sealed abstract class

2022-02-25 Thread Lance Andersen 
On Fri, 25 Feb 2022 19:02:55 GMT, Naoto Sato  wrote:

> Refactoring `java.time.ZoneId` class to be a sealed class. A CSR has also 
> been drafted.

Marked as reviewed by lancea (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/7625

Re: RFR: 8282131: java.time.ZoneId should be a sealed abstract class

2022-02-25 Thread Mandy Chung 
On Fri, 25 Feb 2022 19:02:55 GMT, Naoto Sato  wrote:

> Refactoring `java.time.ZoneId` class to be a sealed class. A CSR has also 
> been drafted.

Thanks for doing this.  Looks good.

-

Marked as reviewed by mchung (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/7625

Re: RFR: 8282131: java.time.ZoneId should be a sealed abstract class

2022-02-25 Thread Brian Burkhalter 
On Fri, 25 Feb 2022 19:02:55 GMT, Naoto Sato  wrote:

> Refactoring `java.time.ZoneId` class to be a sealed class. A CSR has also 
> been drafted.

Marked as reviewed by bpb (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/7625

Re: RFR: 8282131: java.time.ZoneId should be a sealed abstract class

2022-02-25 Thread Roger Riggs 
On Fri, 25 Feb 2022 19:02:55 GMT, Naoto Sato  wrote:

> Refactoring `java.time.ZoneId` class to be a sealed class. A CSR has also 
> been drafted.

Marked as reviewed by rriggs (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/7625

Re: RFR: 8282131: java.time.ZoneId should be a sealed abstract class

2022-02-25 Thread Iris Clark 
On Fri, 25 Feb 2022 19:02:55 GMT, Naoto Sato  wrote:

> Refactoring `java.time.ZoneId` class to be a sealed class. A CSR has also 
> been drafted.

Associated CSR also reviewed.

-

Marked as reviewed by iris (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/7625

RFR: 8282131: java.time.ZoneId should be a sealed abstract class

2022-02-25 Thread Naoto Sato 
Refactoring `java.time.ZoneId` class to be a sealed class. A CSR has also been 
drafted.

-

Commit messages:
 - 8282131: java.time.ZoneId should be a sealed abstract class

Changes: https://git.openjdk.java.net/jdk/pull/7625/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk=7625=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8282131
  Stats: 10 lines in 1 file changed: 0 ins; 4 del; 6 mod
  Patch: https://git.openjdk.java.net/jdk/pull/7625.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/7625/head:pull/7625

PR: https://git.openjdk.java.net/jdk/pull/7625