RFR: 8345286: Remove use of SecurityManager API from misc areas [v2]
    Severin Gehwolf 
    sgehwolf at openjdk.org
       
    Mon Dec  2 13:46:43 UTC 2024
    
    
  
On Mon, 2 Dec 2024 12:43:20 GMT, Alan Bateman <alanb at openjdk.org> wrote:
>> Jaikiran Pai has updated the pull request incrementally with four additional commits since the last revision:
>> 
>>  - remove unnecessary space
>>  - Path.of() instead of Paths.get()
>>  - fix formatting of try-with-resources in CgroupSubsystemController.java
>>  - leave out MethodUtil from the clean up
>
> src/java.base/linux/classes/jdk/internal/platform/CgroupSubsystemController.java line 167:
> 
>> 165:         if (controller == null) return defaultRetval;
>> 166: 
>> 167:         try (Stream<String> lines = Files.lines(Paths.get(controller.path(), param))) {
> 
> Using Path.of might be clearer here.
What Alan said.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22478#discussion_r1865861779
    
    
More information about the serviceability-dev
mailing list