[PATCH FOR REVIEW]: Fix CORBA documentation warnings

[Ken Cavanaugh]

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
>