On Tue, 19 Nov 2024 12:06:51 GMT, Eirik Bjørsnøs <eir...@openjdk.org> wrote:
>> 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. thanks 8-) ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22106#discussion_r1848264566
