[PATCH FOR REVIEW]: Fix CORBA documentation warnings

Joseph D. Darcy Joe.Darcy at Sun.COM
Sat Jan 16 00:09:06 UTC 2010


Andrew John Hughes wrote:
> 2010/1/15 Ken Cavanaugh <Ken.Cavanaugh at sun.com>:
>   
>> The fixes look good to me.
>>
>> Thanks,
>>
>> Ken.
>>
>> On Jan 15, 2010, at 1:59 PM, Joseph D. Darcy wrote:
>>
>>     
>>> Andrew John Hughes wrote:
>>>       
>>>> When building documentation, both OpenJDK6 and OpenJDK7 spit out a
>>>> number of warnings when building documentation:
>>>>
>>>> /mnt/builder/jdk6/impsrc/javax/rmi/PortableRemoteObject.java:171:
>>>> warning - Tag @link: reference not found: Stub#connect
>>>> /mnt/builder/jdk6/impsrc/org/omg/CORBA/SetOverrideType.java:50:
>>>> warning - Tag @link: reference not found: omg.org.CORBA.Object._se\
>>>> t_policy_override
>>>> /mnt/builder/jdk6/impsrc/org/omg/CORBA/TCKind.java:552: warning - Tag
>>>> @return cannot be used in constructor documentation.  It can\
>>>> only be used in the following types of documentation: method.
>>>> /mnt/builder/jdk6/impsrc/org/omg/CORBA/UnknownUserException.java:62:
>>>> warning - @ is an unknown tag.
>>>> /mnt/builder/jdk6/impsrc/org/omg/CORBA/portable/ServantObject.java:48:
>>>> warning - Tag @return cannot be used in field documentation\
>>>> .  It can only be used in the following types of documentation: method.
>>>>
>>>> /mnt/builder/jdk6/impsrc/org/omg/CosNaming/_NamingContextExtStub.java:301:
>>>> warning - @parm is an unknown tag.
>>>> /mnt/builder/jdk6/impsrc/org/omg/CosNaming/_NamingContextStub.java:146:
>>>> warning - @parm is an unknown tag.
>>>>
>>>> /mnt/builder/jdk6/impsrc/org/omg/CosNaming/NamingContextOperations.java:89:
>>>> warning - @parm is an unknown tag.
>>>>
>>>> /mnt/builder/jdk6/impsrc/org/omg/PortableInterceptor/IORInfoOperations.java:54:
>>>> warning - @param argument "a_component" is not a p\
>>>> arameter name.
>>>>
>>>> /mnt/builder/jdk6/impsrc/org/omg/PortableInterceptor/IORInfoOperations.java:72:
>>>> warning - @param argument "a_component" is not a p\
>>>> arameter name.
>>>>
>>>> This patch against OpenJDK7:
>>>>
>>>> http://cr.openjdk.java.net/~andrew/build/webrev.04/corba.patch
>>>>
>>>> fixes the warnings.
>>>>
>>>> Is this ok to push?  If so, can I have a bug ID for it?
>>>>
>>>> Joe, would this also be ok for 6?
>>>>
>>>> Thanks,
>>>>
>>>>         
>>> cc'ing Ken Cavanaugh for corba matters.
>>>
>>> If this fix is approved for JDK 7, I approve it to also go back into
>>> OpenJDK 6.
>>>
>>> Regards,
>>>
>>> -Joe
>>>
>>>       
>>     
>
> Thanks Ken.  I don't see you on http://db.openjdk.java.net/people so
> not sure what I should use for the Reviewed-by field.
>
> Joe, can you allocate this a bug ID?
>   

6917485 Corba doc warnings

You can use me as a reviewer for jcheck purposes.

Have a good weekend,

-Joe

PS Monday is a holiday for Sun in the US.




More information about the build-dev mailing list