<Swing Dev> JDK 9 RFR of JDK-8077095: Fix missing doclint warnings in the javax.swing.plaf.basic package

joe darcy joe.darcy at oracle.com
Wed Apr 8 17:27:48 UTC 2015


Hello Sergey,

On 4/8/2015 7:37 AM, Sergey Bylokhov wrote:
> Hi, Joe.
> What is the benefit of /** {@inheritDoc} */ ?(which are formatted 
> differently in different files) Probably simple @Override will be more 
> helpful?

The essence of this bug fix is adding some explicit javadoc to each 
public or protected declaration. Having "{@inheritDoc}" explicitly says 
"just use the javadoc of some supertype; I have nothing to add," which 
makes the intent clear in the source code even if the generated javadoc 
is the same as if the {@inheritDoc} were omitted and the javadoc default 
action used. (I would prefer if javadoc allowed you to specify which 
supertype the {@inheritDoc} came from, but that RFE has not been 
implemented.)

For the variation in different files, If the code seemed to be using a 
dense style, I tried to use

     /** {@inheritDoc} */

rather than

     /**
      * {@inheritDoc}
     */

to conserve vertical space.

The @Override annotation is related in that the method must be 
overriding one defined in a supertype (and it is good coding practice), 
but doesn't directly interact with the javadoc generation.

>
> BasicFileChooserUI.java: openDialogTitleText/saveDialogTitleText 
> should mention that these are title text.

I'll make that edit.

> BasicInternalFrameUI.java: all text fields is in a lowcase unlike 
> other files, is it expected?

I'll change those to be

      /** Border listener */

rather than

      /** border listener */

etc.

Thanks,

-Joe

>
> On 08.04.15 10:28, joe darcy wrote:
>> Hello,
>>
>> Please review the large but mostly straightforward changes to address:
>>
>>     JDK-8077095: Fix missing doclint warnings in the 
>> javax.swing.plaf.basic package
>>     http://cr.openjdk.java.net/~darcy/8077095.0/
>>
>> This is the last large piece needed to fix the missing doclint in the 
>> client libraries. (Thankfully!)
>>
>> Thanks,
>>
>> -Joe
>
>




More information about the swing-dev mailing list