On Thu, 14 Nov 2024 13:03:28 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.
Marked as reviewed by alanb (Reviewer).
src/java.instrument/share/classes/sun/instrument/InstrumentationImpl.java line
661:
> 659: static final StackWalker walker;
> 660: static {
> 661: walker =
> StackWalker.getInstance(StackWalker.Option.RETAIN_CLASS_REFERENCE);
If you want, you could remove the static block, but what you have is okay too.
-------------
PR Review: https://git.openjdk.org/jdk/pull/22106#pullrequestreview-2441725031
PR Review Comment: https://git.openjdk.org/jdk/pull/22106#discussion_r1846005325
