2010/1/15 Ken Cavanaugh <ken.cavana...@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? Thanks, -- Andrew :-) Free Java Software Engineer Red Hat, Inc. (http://www.redhat.com) Support Free Java! Contribute to GNU Classpath and the OpenJDK http://www.gnu.org/software/classpath http://openjdk.java.net PGP Key: 94EFD9D8 (http://subkeys.pgp.net) Fingerprint: F8EF F1EA 401E 2E60 15FA 7927 142C 2591 94EF D9D8