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

Reply via email to