RFR: 8345286: Remove use of SecurityManager API from misc areas [v8]
Alan Bateman
alanb at openjdk.org
Mon Dec 2 18:57:43 UTC 2024
On Mon, 2 Dec 2024 18:48:27 GMT, Sean Mullan <mullan at openjdk.org> wrote:
>> Jaikiran Pai has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 13 commits:
>>
>> - merge latest from master branch
>> - remove changes to src/java.base/unix/classes/sun/security/provider/NativePRNG.java
>> - remove unused import
>> - replace remaining Paths.get() with Path.of() in the updated files
>> - Update src/java.base/linux/classes/jdk/internal/platform/CgroupSubsystemController.java
>>
>> Co-authored-by: Severin Gehwolf <jerboaa at gmail.com>
>> - Update src/java.base/linux/classes/jdk/internal/platform/cgroupv2/CgroupV2Subsystem.java
>>
>> Co-authored-by: Severin Gehwolf <jerboaa at gmail.com>
>> - Path.of() instead of Paths.get() in CgroupV2Subsystem.java
>> - remove unnecessary space
>> - Path.of() instead of Paths.get()
>> - fix formatting of try-with-resources in CgroupSubsystemController.java
>> - ... and 3 more: https://git.openjdk.org/jdk/compare/30b8bbe2...9baa279b
>
> src/java.base/share/classes/jdk/internal/misc/InnocuousThread.java line 31:
>
>> 29:
>> 30: /**
>> 31: * A thread that has no permissions, is not a member of any user-defined
>
> I think you can also remove the words "has no permissions" as it no longer applies. @AlanBateman ?
Yes, that can be removed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22478#discussion_r1866425533
More information about the serviceability-dev
mailing list