<Swing Dev> RFR: 8251125: doclint errors about missing references in Swing javadoc
Prasanta Sadhukhan
prasanta.sadhukhan at oracle.com
Thu Aug 13 14:12:51 UTC 2020
looks ok to me.
Regards
Prasanta
On 13-Aug-20 7:05 PM, Tejpal Rebari wrote:
> Hi Prasanta,
>
>> On 13-Aug-2020, at 6:52 PM, Prasanta Sadhukhan
>> <prasanta.sadhukhan at oracle.com
>> <mailto: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
>>>> <mailto: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/
>
> 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/dae555a9/attachment-0001.htm>
More information about the swing-dev
mailing list