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

[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.

While you're at it maybe you should remove the strange <br> 
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      * <br> 

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
>