On Thu, 24 Oct 2024 20:57:50 GMT, Sean Mullan <mul...@openjdk.org> wrote:

>> @honkar-jdk I'm inclined to leave it as because it's not the only method 
>> which doesn't have a blank line between `@param` and `@throw` in this file.
>> 
>> If it's worth taking care of, we may submit another bug to address it later.
>
> This does not need to be handled in this PR. In the majority of changes, we 
> have really tried hard to avoid making unrelated changes, but sometimes a few 
> snuck in, like moving package imports or perhaps fixing a typo here and there 
> that was not specific to JEP 486. My opinion is that unless it is something 
> that _really_ should be done as part of a more general technical debt or code 
> cleanup exercise, then it is ok to let a few of these in and they don't have 
> to be reverted.

> > Can we add a blank line here? It's present in the methods above.
> > 
> > Although there are other places below where it's missing; _so not worth 
> > worrying_.
>
> … it's not the only method which doesn't have a blank line between `@param` 
> and `@throw` in this file.
> 
> If it's worth taking care of, we may submit another bug to address it later.

I've submitted [JDK-8343448](https://bugs.openjdk.org/browse/JDK-8343448): 
_Improve formatting of docs in java.awt.Desktop_.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21498#discussion_r1826079117

Reply via email to