<Swing Dev> JDK 9 RFR of JDK-8077095: Fix missing doclint warnings in the javax.swing.plaf.basic package
Joseph D. Darcy
joe.darcy at oracle.com
Fri Apr 10 01:28:31 UTC 2015
PS On a somewhat related note, if all the client doclint fixes currently
out for review are applied to a forest, the "missing" doclint check can
be enabled for the java.desktop module:
http://mail.openjdk.java.net/pipermail/build-dev/2015-April/014749.html
In other words, assuming no new missing javadoc is introduced, this
should be last of the missing doclint fixes :-)
Thanks,
-Joe
On 4/8/2015 10:27 AM, joe darcy wrote:
> 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