RFR: 6955128: Spec for javax.swing.plaf.basic.BasicTextUI.getVisibleEditorRect contains inappropriate wording [v2]

Prasanta Sadhukhan psadhukhan at openjdk.org
Wed Jun 25 12:15:55 UTC 2025


On Wed, 25 Jun 2025 10:47:27 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> We may break the description into paragraphs like this:
>> 
>> 
>>     /**
>>      * Gets the allocation for the root view.
>>      * <p>
>>      * The returned rectangle is unrelated to visibility.
>>      * It is used to set the size of the root view.
>>      * <p>
>>      * The component must have a non-zero positive size for
>>      * this translation to be computed.
>>      *
>>      * @return the bounding box for the root view
>>      *
>>      * @see View#paint
>>      * @see View#setSize
>>      */
>>     protected Rectangle getVisibleEditorRect() {
>> 
>> This way different parts of the description are clearly separated, therefore each part stands out when one scans the javadoc for information.
>
> If linked with ‘and’, there should be no comma.

ok..modified..

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25850#discussion_r2166566514


More information about the client-libs-dev mailing list