Core Libs Dev[9] Review request for 8039041: tidy warnings cleanup for javax.naming
Hello, Could you please review the fix for the following bug: https://bugs.openjdk.java.net/browse/JDK-8039041 Webrev corresponding: http://cr.openjdk.java.net/~yan/8039041/webrev.00/ Just a minor cleanup of javadoc to avoid tidy warnings; no other code affected. Thanks. Regards, Alexander
Re: Core Libs Dev[9] Review request for 8039041: tidy warnings cleanup for javax.naming
Hello, I think doclint should checks this type of issues? On 03.04.2014 13:01, alexander stepanov wrote: Hello, Could you please review the fix for the following bug: https://bugs.openjdk.java.net/browse/JDK-8039041 Webrev corresponding: http://cr.openjdk.java.net/~yan/8039041/webrev.00/ Just a minor cleanup of javadoc to avoid tidy warnings; no other code affected. Thanks. Regards, Alexander -- Best regards, Sergey.
Re: Core Libs Dev[9] Review request for 8039041: tidy warnings cleanup for javax.naming
Looks fine Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com Sent from my iPad On Apr 3, 2014, at 5:01 AM, alexander stepanov alexander.v.stepa...@oracle.com wrote: Hello, Could you please review the fix for the following bug: https://bugs.openjdk.java.net/browse/JDK-8039041 Webrev corresponding: http://cr.openjdk.java.net/~yan/8039041/webrev.00/ Just a minor cleanup of javadoc to avoid tidy warnings; no other code affected. Thanks. Regards, Alexander
Re: Core Libs Dev[9] Review request for 8039041: tidy warnings cleanup for javax.naming
On 03/04/2014 10:01, alexander stepanov wrote: Hello, Could you please review the fix for the following bug: https://bugs.openjdk.java.net/browse/JDK-8039041 Webrev corresponding: http://cr.openjdk.java.net/~yan/8039041/webrev.00/ Just a minor cleanup of javadoc to avoid tidy warnings; no other code affected. This looks okay to me. -Alan.