On Mon, 18 Nov 2024 20:28:49 GMT, Brian Burkhalter <b...@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

Reply via email to