[16] RFR: 8251182: Fix "no comment" warnings in java.naming
Lance Andersen
LANCE.ANDERSEN at ORACLE.COM
Wed Aug 26 15:59:34 UTC 2020
+1
> On Aug 26, 2020, at 7:46 AM, Aleks Efimov <aleksej.efimov at oracle.com> wrote:
>
> Hi Roger, Lance, Daniel,
>
> Thanks for your reviews and comments!
> New webrev with suggested modifications can be viewed at this location:
> http://cr.openjdk.java.net/~aefimov/8251182/01
>
> The list of modifications:
> - Added @java.io.Serial annotation to 'serialVersionUID' declarations
> - Removed trailing periods from @param/@throws
> - Added generic first line comments to writeObject/readObject methods
>
> The new version of patch and webrev files have been uploaded to the CSR issue.
>
> Kind Regards,
> Aleksei
>
> On 25/08/2020 20:23, Daniel Fuchs wrote:
>> Hi Aleksei,
>>
>> LGTM.
>>
>> You could also add the @Seial annotation to serialVersionUID
>> declarations.
>>
>> best regards,
>>
>> -- daniel
>>
>> On 25/08/2020 16:03, Aleks Efimov wrote:
>>> Hi,
>>>
>>> The documentation of classes from "java.naming" module needs to be improved to resolve javadoc -Xdoclint "no comment" warnings.
>>>
>>> JBS issue: https://bugs.openjdk.java.net/browse/JDK-8251182
>>> Webrev: http://cr.openjdk.java.net/~aefimov/8251182/00/index.html
>>> CSR: https://bugs.openjdk.java.net/browse/JDK-8252310
>>>
>>> With Best Regards,
>>> Aleksei
>>
>
Best
Lance
------------------
Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
Lance.Andersen at oracle.com
More information about the core-libs-dev
mailing list