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