javadoc warning fixes

Alan Bateman Alan.Bateman at Sun.COM
Wed Jul 29 13:34:27 PDT 2009


Martin Buchholz wrote:
> On Wed, Jul 29, 2009 at 12:46, Alan Bateman<Alan.Bateman at sun.com> wrote:
>   
>> Martin Buchholz wrote:
>>     
>>> Here are some easy javadoc warning fixes:
>>>
>>> http://cr.openjdk.java.net/~martin/webrevs/openjdk7/nio-javadoc-warnings/
>>>
>>>       
>> Thanks for catching these.
>>
>> The link to bind (in DatagramChannel) is already fixed in the nio
>> repository, I just hadn't got around to pushing it. No problem if fix it in
>> jdk7/tl/jdk first.
>>     
>
> OK.
>
>   
>> In the attribute/package-info.java I think I prefer "lookup" because that is
>> consistent with the class description in UserPrincipalLookupService. I agree
>>     
>
> OK.
>
>   
>> with the other fixes (the wrong package for the links to CME is kinda
>> embarrassing).
>>
>> Anyway, I've created so you can push these:
>>  6866554: Javadoc warnings in channels and file packages
>>
>>     
>>> Here are more unfixed warnings in nio-land that I'm not addressing:
>>>
>>> ../../src/share/classes/java/nio/file/FileRef.java:48: warning - Tag
>>> @see: reference not found: java.io.Inputs
>>> ../../src/share/classes/java/nio/file/FileRef.java:48: warning - Tag
>>> @see: reference not found: java.io.Outputs
>>> ../../src/share/classes/java/nio/channels/package-info.java:293:
>>> warning - Tag @link: reference not found: java.nio.MappedBigByteBuffer
>>>
>>>       
>> The link to the non-existent MappedBigByteBuffer is fixed in the nio
>> repository (someone reported a few months ago).  Can you include it in your
>> changes?
>>     
>
> Done.
>
> Martin
>   
Updated webrev looks good - thanks for taking the 
channels/package-info.java update.

-Alan.



More information about the nio-dev mailing list