RFR: 8344187: Remove SecurityManager and related calls from java.lang.instrument

[Kevin Walls]

On Thu, 14 Nov 2024 14:53:44 GMT, Alan Bateman <alanb at 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.
>
> src/java.instrument/share/classes/sun/instrument/InstrumentationImpl.java line 484:
> 
>> 482:     // reflected object (for example, a method)
>> 483:     private static void setAccessible(final AccessibleObject ao, final boolean accessible) {
>> 484:         ao.setAccessible(accessible);
> 
> I think you can remove this method, and instead change the usage in loadClassAndStartAgent to just call setAccessible.

will do, yes it is trivial now.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/22106#discussion_r1842367446