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

Iris Clark iris at openjdk.org
Mon Nov 25 19:13:19 UTC 2024


On Thu, 21 Nov 2024 20:41:36 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:
> 
>   spacing changes

src/java.prefs/macosx/classes/java/util/prefs/MacOSXPreferencesFile.java line 87:

> 85:     @SuppressWarnings("restricted")
> 86:     private static void loadPrefsLib() {
> 87:         System.loadLibrary("prefs");

What am I missing?  Why keep the private static method if it only contains a single method invocation and is only used in the static initializer?

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

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


More information about the core-libs-dev mailing list