RFR: 8344181: Remove SecurityManager and related calls from jdk.management and jdk.management.agent [v5]

Kevin Walls kevinw at openjdk.org
Mon Nov 25 12:43:22 UTC 2024


On Thu, 21 Nov 2024 10:17:36 GMT, Kevin Walls <kevinw at openjdk.org> wrote:

>> Remove redundant SecurityManager, AccessController references
>> (following on from JDK-8338411: Implement JEP 486: Permanently Disable the Security Manager).
>> 
>> src/jdk.management/share/classes/com/sun/management/internal/GarbageCollectionNotifInfoCompositeData.java
>> There is an existing theoretical path where GcInfoBuilder stays null, should never happen, "com.sun.management.GcInfo" exists...
>> 
>> src/jdk.management/share/classes/com/sun/management/internal/GcInfoCompositeData.java
>> Similarly there is an existing assumption that Class.forName("com.sun.management.GcInfo") succeeds.
>
> Kevin Walls has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - newline
>  - remove implSetParallelism from VSThreadSchedulers

Thanks for the reviews!

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

PR Comment: https://git.openjdk.org/jdk/pull/22155#issuecomment-2497915512


More information about the serviceability-dev mailing list