RFR: JDK-8283416: Update java.lang.invoke.MethodHandle to use sealed classes [v2]

Joe Darcy darcy at openjdk.java.net
Thu Mar 24 18:58:43 UTC 2022


> Small refactoring to use sealed classes in the MethodHandle implementation hierarchy.
> 
> DelegatingMethodHandle is non-sealed rather than sealed since it has two subclasses, one defined as a nested class and only a separate type in the same package. The permits clause does not allow listed those two kinds of subclasses.
> 
> Please also review the corresponding CSR https://bugs.openjdk.java.net/browse/JDK-8283434

Joe Darcy has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:

 - Respond to review feedback.
 - Merge branch 'master' into JDK-8283416
 - JDK-8283416: Update java.lang.invoke.MethodHandle to use sealed classes

-------------

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/7881/files
  - new: https://git.openjdk.java.net/jdk/pull/7881/files/f4f79411..e7c6495f

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7881&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7881&range=00-01

  Stats: 65233 lines in 1034 files changed: 62326 ins; 1345 del; 1562 mod
  Patch: https://git.openjdk.java.net/jdk/pull/7881.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/7881/head:pull/7881

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


More information about the core-libs-dev mailing list