> On Jun 4, 2018, at 12:00 PM, Lance Andersen <lance.ander...@oracle.com> wrote:
>
>
>> On Jun 4, 2018, at 2:51 PM, Paul Sandoz <paul.san...@oracle.com
>> <mailto:paul.san...@oracle.com>> wrote:
>>
>>
>>
>>> On Jun 4, 2018, at 4:22 AM, Lance Andersen <lance.ander...@oracle.com
>>> <mailto:lance.ander...@oracle.com>> wrote:
>>>
>>> Hi,
>>>
>>> Bug 8201608 highlights a few broken links in javax/sql/rowset/package.html
>>> and javax/sql/rowset/spi/package.html
>>>
>>> As part of this fix, I took the liberty to move from package.html to
>>> package-info.java
>>>
>>> The webrev can be found at
>>> http://cr.openjdk.java.net/~lancea/8201608/webrev.00/
>>> <http://cr.openjdk.java.net/~lancea/8201608/webrev.00/>
>>> <http://cr.openjdk.java.net/~lancea/8201608/webrev.00/
>>> <http://cr.openjdk.java.net/~lancea/8201608/webrev.00/>>
>>>
>>> I have also attached a diff of the changes as it is less obvious of the
>>> webrev prior to the migration to package-info.java
>>>
>>
>> The attachment got dropped. Upload it to cr?
>
> Sorry about that, forgot that attachments are stripped. It is attached now
> to https://bugs.openjdk.java.net/browse/JDK-8201608
> <https://bugs.openjdk.java.net/browse/JDK-8201608>
>
Thanks, +1 (from mostly reviewing the diff and not the package-info.java
itself).
Paul.