RFR: 8344181: Remove SecurityManager and related calls from jdk.management and jdk.management.agent [v5]
Alex Menkov
amenkov at openjdk.org
Fri Nov 22 19:10:29 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
Marked as reviewed by amenkov (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/22155#pullrequestreview-2455435291
More information about the serviceability-dev
mailing list