On Thu, 23 Nov 2023 15:55:42 GMT, Jim Laskey <jlas...@openjdk.org> wrote:
>> Address changes from JEP 445 to JEP 463. >> >> - Move from a SYNTHETIC unnamed class to a MANDATED implicit class. >> >> - Don't mark class on read. >> >> - Remove reflection and annotation processing related to unnamed classes. >> >> - Simplify main method search. > > Jim Laskey has updated the pull request incrementally with one additional > commit since the last revision: > > Remove Test src/java.base/share/classes/jdk/internal/misc/MethodFinder.java line 40: > 38: } > 39: > 40: private static final JavaLangAccess JLA = > SharedSecrets.getJavaLangAccess(); The new shared secret and usage looks fine. It would be good to add a short class description and method description and make it clear it's for launcher usage, we don't want this used for anything else. Personally I would move JLA to to the top rather than after the constructor. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16461#discussion_r1404252179