RFR: 8344337: SecurityManager cleanup in java.prefs module [v4]

Brian Burkhalter bpb at openjdk.org
Wed Nov 20 18:49:21 UTC 2024


On Wed, 20 Nov 2024 18:03:53 GMT, Brent Christian <bchristi at openjdk.org> wrote:

>> Remove usages of SecurityManager, doPrivildged, and AccessController from the java.prefs module.
>
> Brent Christian has updated the pull request incrementally with one additional commit since the last revision:
> 
>   fix indentation, add braces

src/java.prefs/unix/classes/java/util/prefs/FileSystemPreferences.java line 127:

> 125:                 getLogger().info("Created user preferences directory.");
> 126:             }
> 127:             else

It seems odd that `else` is not on line 126 after the bracket. If putting it on the same line makes it  less readable, then maybe add a pair of brackets for the `else`.

src/java.prefs/unix/classes/java/util/prefs/FileSystemPreferences.java line 586:

> 584:                 m = new TreeMap<>();
> 585:             } else if (e instanceof FileNotFoundException) {
> 586:             getLogger().warning("Prefs file removed in background "

This could be indented.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22252#discussion_r1850819795
PR Review Comment: https://git.openjdk.org/jdk/pull/22252#discussion_r1850816922


More information about the core-libs-dev mailing list