On Mon, 18 Nov 2024 10:50:24 GMT, Kevin Walls <kev...@openjdk.org> wrote:
>> Remove redundant SecurityManager, AccessController references >> (following on from JDK-8338411: Implement JEP 486: Permanently Disable the >> Security Manager). >> >> test/jdk/java/lang/instrument/ still passing. > > Kevin Walls 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 four additional > commits since the last revision: > > - static block not needed > - Merge remote-tracking branch 'upstream/master' into > 8344187_PostSM_instrument > - update > - 8344187: Remove SecurityManager and related calls from java.instrument src/java.instrument/share/classes/sun/instrument/InstrumentationImpl.java line 40: > 38: import java.net.URL; > 39: import java.security.CodeSource; > 40: import java.security.PrivilegedAction; A note for future cleanups in this area: This `java.security.PrivilegedAction` import should probably have been removed. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22106#discussion_r1848231895
