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

Daniel Fuchs dfuchs at openjdk.org
Mon Nov 25 17:09:24 UTC 2024


On Mon, 25 Nov 2024 16:22:16 GMT, Sean Mullan <mullan 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/LazyLoggers.java line 346:
> 
>> 344:             // the result.
>> 345:             // This is just an optimization to avoid the cost of calling
>> 346:             // doPrivileged every time.
> 
> These 2 lines can be removed.

darn. I thought I had removed everything. Noted. thanks for noticing!

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

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


More information about the core-libs-dev mailing list