RFR: 8344077: Remove security manager dependency in java.io
Roger Riggs
rriggs at openjdk.org
Mon Nov 18 20:56:51 UTC 2024
On Mon, 18 Nov 2024 20:28:49 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:
> Expunge the use of the `SecurityManager`, `AccessController`, and the like from the `java.io` package.
Looks good
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());
-------------
Marked as reviewed by rriggs (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/22219#pullrequestreview-2443700481
PR Review Comment: https://git.openjdk.org/jdk/pull/22219#discussion_r1847254590
More information about the core-libs-dev
mailing list