RFR: 8349206: j.u.l.Handler classes create deadlock risk via synchronized publish() method [v5]

[David Beaumont]

On Mon, 24 Feb 2025 12:19:19 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> David Beaumont has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Adding @implNote to new JavaDoc.
>
> src/java.logging/share/classes/java/util/logging/Handler.java line 47:
> 
>> 45:  *
>> 46:  * <h2>Subclass Implementation Notes</h2>
>> 47:  *
> 
> Have you looked at how this is rendered in the generated API documentation when loaded up in the browser?
> 
> I mean `@implNote` is usually a section on its own, so I am unsure whether lines 46-47 above should be moved inside the `@implNote` section or simply removed. Or maybe it's OK to leave them there - but I am afraid it would appear as an empty subsection in the normative API documentation.

You're right, it looks wrong with the `<h2>` and the @implNote. I'll just remove the `<h2>`.
I hadn't appreciated that the note would have its own section. Thanks for spotting.
It also seems that you can only add one annotation per JavaDoc entry, so I removed the 2nd.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23491#discussion_r1967604106