RFR: 8344077: Remove security manager dependency in java.io [v2]

Brian Burkhalter bpb at openjdk.org
Mon Nov 18 21:08:00 UTC 2024


On Mon, 18 Nov 2024 20:46:24 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> Brian Burkhalter has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - 8344446: Address review comments
>>  - 8344446: Remove security manager dependency from module jdk.sctp
>
> src/java.base/share/classes/java/io/FilePermission.java line 184:
> 
>> 182:         DefaultFileSystemProvider.theFileSystem();
>> 183: 
>> 184:     private static final Path here = builtInFS.getPath(System.getProperty("user.dir"));
> 
> Could be updated to use StaticProperty:
> Suggestion:
> 
>     private static final Path here = builtInFS.getPath(jdk.internal.util.StaticProperty.userDir());

So changed in c7ba335.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22219#discussion_r1847275036


More information about the core-libs-dev mailing list