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

[alexander stepanov]

Hello,

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

 > I just assume the removal of the </p> 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 <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
>>
>