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

[Jaroslav Bachorik]

Hi Alexander,

On 1.4.2014 15: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

src/share/classes/javax/management/remote/rmi/package.html; line 248 - 
</p> tag could be removed too to make the changes consistent.

-JB-

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