Re: jmx-dev Core Libs Dev[9] Review Request for 8038795: tidy warnings cleanup for javax.management

2014-04-01 Thread Daniel Fuchs
Hi Alexander, Looks good. I think I would remove the leading p 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.

Re: jmx-dev Core Libs Dev[9] Review Request for 8038795: tidy warnings cleanup for javax.management

2014-04-01 Thread Daniel Fuchs
On 4/1/14 4:41 PM, alexander stepanov wrote: Hello, Please see the updated webrev here: http://cr.openjdk.java.net/~yan/8038795/webrev.01/ Thanks Alexander, It looks good! -- daniel I just assume the removal of the /p isn't really necessary. Yes, probably. That was done just to make