jmx-dev [9] Review Request for 8038795: tidy warnings cleanup for javax.management (original) (raw)

alexander stepanov alexander.v.stepanov at oracle.com
Tue Apr 1 14:41:32 UTC 2014

Hello,

Please see the updated webrev here: http://cr.openjdk.java.net/~yan/8038795/webrev.01/

I just assume the removal of the

isn't really necessary. Yes, probably. That was done just to make tidy happy.

Thanks, Alexander

On 01.04.2014 17:44, Daniel Fuchs wrote:

Hi Alexander,

Looks good. I think I would remove the leading

in

http://cr.openjdk.java.net/~yan/8038795/webrev.00/src/share/classes/javax/management/remote/JMXPrincipal.java.frames.html as well. If you don't please check that the generated javadoc for JMXPrincipal.java still looks good. While you're at it maybe you should remove the strange
  between the @param tags in <http://cr.openjdk.java.net/~yan/8038795/webrev.00/src/share/classes/javax/management/openmbean/CompositeType.java.frames.html>

such as: 86 *
  best regards, -- daniel On 4/1/14 2:57 PM, Alan Bateman wrote:

I think you are looking for jmx-dev so forwarding to that list. On 01/04/2014 13:51, alexander stepanov wrote: Hello,

Could you please review the fix for the following bug: https://bugs.openjdk.java.net/browse/JDK-8038795 Webrev corresponding: http://cr.openjdk.java.net/~yan/8038795/webrev.00/ Just a minor cleanup of javadoc to avoid tidy warnings; no other code affected. Thanks. Regards, Alexander

More information about the core-libs-dev mailing list