On Tue, 8 Apr 2025 16:15:13 GMT, Chen Liang <li...@openjdk.org> wrote:
>> 8353683: j.u.l.Handler classes create deadlock risk via synchronized
>> publish() method.
>>
>> 1. Remove synchronization of calls to publish() in Handlers in
>> java.util.logging package.
>> 2. Add explanatory comments to various affected methods.
>> 3. Add a test to ensure deadlocks no longer occur.
>>
>> Note that this change does not address issue in MemoryHandler (see
>> JDK-8349208).
>
> src/java.logging/share/classes/java/util/logging/Handler.java line 46:
>
>> 44: * {@code Handler} class.
>> 45: *
>> 46: * <h2><a id=threadSafety>Thread Safety and Deadlock Risk in
>> Handlers</a></h2>
>
> FYI you can add the id attribute to the h2 tag without declaring a separate a
> tag.
I'm copying exactly what Joe suggested, so I'll leave it like this for
consistency. But thanks for pointing that out.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24504#discussion_r2035486048
