On Thu, 28 Nov 2024 17:18:36 GMT, Alan Bateman <al...@openjdk.org> wrote:
>> Sean Mullan has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Removal of unused fields from AccessControlContext - no longer needed now >> that VM cleanup is done. > > src/java.base/share/classes/sun/security/util/SecurityConstants.java line 61: > >> 59: // Permission constants used in the various checkPermission() calls >> in JDK. >> 60: >> 61: // java.lang.Class, java.net.URLConnection, >> java.security.AllPermission > > BTW: It's okay to change the usage in Class.Holder.<clinit> to use `new > AllPermission()`. That will help get to the point where this file can be > removed. Fixed in https://github.com/openjdk/jdk/pull/22418/commits/5ad5e8c8b13e3aa8f8379def6b3bb3e994fd084d. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22418#discussion_r1866060865
