Re: RFR: 8283470: Update java.lang.invoke.VarHandle to use sealed classes

2022-03-29 Thread Paul Sandoz 
On Wed, 23 Mar 2022 16:27:29 GMT, Mandy Chung  wrote:

> This patch changes VarHandle and its implementation hierarchy to use sealed 
> classes.  All VarHandle permitted classes are package-private and either 
> final or sealed abstract classes.
> 
> Please also review the CSR: https://bugs.openjdk.java.net/browse/JDK-8283540

Marked as reviewed by psandoz (Reviewer).

-

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

Re: RFR: 8283470: Update java.lang.invoke.VarHandle to use sealed classes

2022-03-23 Thread Joe Darcy 
On Wed, 23 Mar 2022 16:27:29 GMT, Mandy Chung  wrote:

> This patch changes VarHandle and its implementation hierarchy to use sealed 
> classes.  All VarHandle permitted classes are package-private and either 
> final or sealed abstract classes.
> 
> Please also review the CSR: https://bugs.openjdk.java.net/browse/JDK-8283540

Looks fine.

-

Marked as reviewed by darcy (Reviewer).

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

RFR: 8283470: Update java.lang.invoke.VarHandle to use sealed classes

2022-03-23 Thread Mandy Chung 
This patch changes VarHandle and its implementation hierarchy to use sealed 
classes.  All VarHandle permitted classes are package-private and either final 
or sealed abstract classes.

Please also review the CSR: https://bugs.openjdk.java.net/browse/JDK-8283540

-

Commit messages:
 - 8283470: Update java.lang.invoke.VarHandle to use sealed classes

Changes: https://git.openjdk.java.net/jdk/pull/7926/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk=7926=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8283470
  Stats: 58 lines in 5 files changed: 42 ins; 5 del; 11 mod
  Patch: https://git.openjdk.java.net/jdk/pull/7926.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/7926/head:pull/7926

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