<Swing Dev> RFR: 8251125: doclint errors about missing references in Swing javadoc

Tejpal Rebari tejpal.rebari at oracle.com
Thu Aug 13 13:35:49 UTC 2020


Hi Prasanta,

> On 13-Aug-2020, at 6:52 PM, Prasanta Sadhukhan <prasanta.sadhukhan at oracle.com> wrote:
> 
> 
> On 13-Aug-20 6:46 PM, Tejpal Rebari wrote:
>> Hi Prasanta,
>> Thanks for the review.
>> 
>>> On 13-Aug-2020, at 6:36 PM, Prasanta Sadhukhan <prasanta.sadhukhan at oracle.com> wrote:
>>> 
>>> Looks ok to me with one request to not break package-info l110-111 lines and make it into 1 line
>> Does it mean 80chars per line doesn’t apply to the documentation part of the file.
>> I have seen that there are other lines in the same file that doesn’t follow 80 chars per line.
> 
> It should. What I mean is dont break abruptly, break at 80chars.
Ok, I have updated the webrev.
 http://cr.openjdk.java.net/~trebari/swing/8251125/webrev01/ <http://cr.openjdk.java.net/~trebari/swing/8251125/webrev01/>

Regards
Tejpal


> Regards
> 
> Prasanta
> 
>> Regards
>> Tejpal
>> 
>>> Regards
>>> 
>>> Prasanta
>>> 
>>> On 13-Aug-20 6:12 PM, Tejpal Rebari wrote:
>>>> Hi All,
>>>> Please review the following fix for jdk16.
>>>> 
>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8251125
>>>> Webrev : http://cr.openjdk.java.net/~trebari/swing/8251125/webrev00/
>>>> 
>>>> Issue : Doclint errors about missing references in Swing javadoc.
>>>> Doclint identified that some of the swing classes are missing reference.
>>>> 
>>>> Fix : Added complete path name where the reference was missing.
>>>> Verified that the doclint doesn’t throw any error after the fix.
>>>> 
>>>> 
>>>> Thanks
>>>> Tejpal

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/swing-dev/attachments/20200813/e6897619/attachment.htm>


More information about the swing-dev mailing list