Core Libs Dev[9] Review request for 8039041: tidy warnings cleanup for javax.naming

2014-04-03 Thread alexander stepanov
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

2014-04-03 Thread Sergey Bylokhov
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/

Re: Core Libs Dev[9] Review request for 8039041: tidy warnings cleanup for javax.naming

2014-04-03 Thread Lance @ Oracle
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

Re: Core Libs Dev[9] Review request for 8039041: tidy warnings cleanup for javax.naming

2014-04-03 Thread Alan Bateman
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