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 ™ 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>
[email protected] <mailto:[email protected]>
Sent from my iPad
On Apr 9, 2014, at 5:45 AM, alexander stepanov
<[email protected]
<mailto:[email protected]>> 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
