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

2024-11-21 Thread Alan Bateman
On Thu, 21 Nov 2024 20:46:47 GMT, Brent Christian wrote: >> src/java.prefs/unix/classes/java/util/prefs/FileSystemPreferences.java line >> 451: >> >>> 449: // Add shutdown hook to flush cached prefs on normal >>> termination >>> 450: Runtime.getRuntime().addShutdownHook( >>> 45

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

2024-11-21 Thread Brent Christian
On Thu, 21 Nov 2024 15:05:44 GMT, Roger Riggs wrote: >> Brent Christian has updated the pull request incrementally with one >> additional commit since the last revision: >> >> indentation fixes > > src/java.prefs/unix/classes/java/util/prefs/FileSystemPreferences.java line > 451: > >> 449:

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

2024-11-21 Thread Brent Christian
On Thu, 21 Nov 2024 06:50:42 GMT, Andrey Turbanov wrote: >> Brent Christian has updated the pull request incrementally with one >> additional commit since the last revision: >> >> indentation fixes > > src/java.prefs/unix/classes/java/util/prefs/FileSystemPreferences.java line > 577: > >> 5

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

2024-11-21 Thread Roger Riggs
On Wed, 20 Nov 2024 22:43:44 GMT, Brent Christian 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: > > indentation

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

2024-11-20 Thread Andrey Turbanov
On Wed, 20 Nov 2024 22:43:44 GMT, Brent Christian 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: > > indentation

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

2024-11-20 Thread Andrey Turbanov
On Wed, 20 Nov 2024 22:43:44 GMT, Brent Christian 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: > > indentation

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

2024-11-20 Thread Brian Burkhalter
On Wed, 20 Nov 2024 22:43:44 GMT, Brent Christian 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: > > indentation

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

2024-11-20 Thread Brent Christian
> 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: indentation fixes - Changes: - all: https://git.openjdk.org/jdk/p