[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