On Fri, 4 Jul 2025 03:09:32 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> 
wrote:

>> BasicTextUI.getVisibleEditorRect wording is rephrased to remove the wording 
>> "Due to an unfortunate set of historical events this method is 
>> inappropriately named".
>
> Prasanta Sadhukhan has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Update javadoc

Changes requested by aivanov (Reviewer).

src/java.desktop/share/classes/javax/swing/plaf/basic/BasicTextUI.java line 
1032:

> 1030: 
> 1031:     /**
> 1032:      * Gets the allocation (i.e. the allocated size) for the root view.

> What I've been asking for is something like\
> "Gets the allocation (i.e. the allocated size) for the root view."

"Gets the allocation (i.e. the size) for the root view."

@prrace Does it sound better?

The [style 
guide](https://www.oracle.com/technical-resources/articles/java/javadoc-tool.html#styleguide)
 for writing javadoc says:

> **Avoid Latin**
> 
> use "also known as" instead of "aka", use "that is" or "to be specific" 
> instead of "i.e.", use "for example" instead of "e.g.", and use "in other 
> words" or "namely" instead of "viz."

Taking the style guide into account, the text should change to

Suggestion:

     * Gets the allocation (that is the allocated size) for the root view.


I still don't like repeating _“allocated”_, does it clarify the meaning of just 
_“the size”_?

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

PR Review: https://git.openjdk.org/jdk/pull/25850#pullrequestreview-2986680868
PR Review Comment: https://git.openjdk.org/jdk/pull/25850#discussion_r2185065187

Reply via email to