Core Libs Dev[9] Review Request for 8039488: tidy warnings cleanup for javax.sql
Hello, Could you please review the fix for the following bug: https://bugs.openjdk.java.net/browse/JDK-8039488 Webrev corresponding: http://cr.openjdk.java.net/~yan/8039488/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 8039488: tidy warnings cleanup for javax.sql
I think this looks ok but a couple of suggestions src/share/classes/javax/sql/rowset/spi/package.html - The URL links at the bottom of the page need updated, please address them as part of this putback if possible src/share/classes/javax/sql/package.html - I would use trade; as part of the change on lines 40 and 42 Best Lance 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 9, 2014, at 5:45 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-8039488 Webrev corresponding: http://cr.openjdk.java.net/~yan/8039488/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 8039488: tidy warnings cleanup for javax.sql
Hello Lance, Thanks; please see the updated webrev: http://cr.openjdk.java.net/~yan/8039488/webrev.01/ Regards, Alexander On 09.04.2014 14:10, Lance @ Oracle wrote: I think this looks ok but a couple of suggestions *src/share/classes/javax/sql/rowset/spi/package.html - *The URL links at the bottom of the page need updated, please address them as part of this putback if possible * * *src/share/classes/javax/sql/package.html *- I would use trade; as part of the change on lines 40 and 42 Best Lance Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 tel:+1.781.442.2037 Oracle Java Engineering 1 Network Drive x-apple-data-detectors://34/0 Burlington, MA 01803 x-apple-data-detectors://34/0 lance.ander...@oracle.com mailto:lance.ander...@oracle.com Sent from my iPad On Apr 9, 2014, at 5:45 AM, alexander stepanov alexander.v.stepa...@oracle.com mailto:alexander.v.stepa...@oracle.com wrote: Hello, Could you please review the fix for the following bug: https://bugs.openjdk.java.net/browse/JDK-8039488 Webrev corresponding: http://cr.openjdk.java.net/~yan/8039488/webrev.00/ http://cr.openjdk.java.net/%7Eyan/8039488/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 8039488: tidy warnings cleanup for javax.sql
Hi Alexander, looks fine. Best Lance On Apr 9, 2014, at 7:58 AM, alexander stepanov alexander.v.stepa...@oracle.com wrote: Hello Lance, Thanks; please see the updated webrev: http://cr.openjdk.java.net/~yan/8039488/webrev.01/ Regards, Alexander On 09.04.2014 14:10, Lance @ Oracle wrote: I think this looks ok but a couple of suggestions *src/share/classes/javax/sql/rowset/spi/package.html - *The URL links at the bottom of the page need updated, please address them as part of this putback if possible * * *src/share/classes/javax/sql/package.html *- I would use trade; as part of the change on lines 40 and 42 Best Lance Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 tel:+1.781.442.2037 Oracle Java Engineering 1 Network Drive x-apple-data-detectors://34/0 Burlington, MA 01803 x-apple-data-detectors://34/0 lance.ander...@oracle.com mailto:lance.ander...@oracle.com Sent from my iPad On Apr 9, 2014, at 5:45 AM, alexander stepanov alexander.v.stepa...@oracle.com mailto:alexander.v.stepa...@oracle.com wrote: Hello, Could you please review the fix for the following bug: https://bugs.openjdk.java.net/browse/JDK-8039488 Webrev corresponding: http://cr.openjdk.java.net/~yan/8039488/webrev.00/ http://cr.openjdk.java.net/%7Eyan/8039488/webrev.00/ Just a minor cleanup of javadoc to avoid tidy warnings; no other code affected. Thanks. Regards, Alexander Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com