RFR: 8344235: Revisit SecurityManager usage in java.logging after JEP 486 and JEP 491 integration [v2]

Daniel Fuchs dfuchs at openjdk.org
Thu Nov 21 10:24:29 UTC 2024


On Wed, 20 Nov 2024 20:29:06 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> Daniel Fuchs has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Review feedback
>
> src/java.base/share/classes/jdk/internal/logger/LoggerFinderLoader.java line 114:
> 
>> 112: 
>> 113:     private static Iterator<System.LoggerFinder> findLoggerFinderProviders() {
>> 114:         final Iterator<System.LoggerFinder> iterator =
> 
> Could be simply be `return ` ... no need for a local var.

Done

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22281#discussion_r1851774290


More information about the core-libs-dev mailing list