Hi Roger > On Jun 6, 2018, at 11:26 AM, Roger Riggs <roger.ri...@oracle.com> wrote: > > Hi Lance, > > That's fine, the conversion from .html to .java made the diff extensive; > hiding the original link fix.
Yes, I had attached them originally to the email with the diff against package.html and then forgot it would be stripped by the mail server so I attached it to the bug per Paul’s suggestion. > The changes are fine by me. Thank you Best Lance > > Roger > > > On 6/6/18 10:56 AM, Lance Andersen wrote: >> Hi Roger >>> On Jun 5, 2018, at 5:54 PM, Roger Riggs <roger.ri...@oracle.com >>> <mailto:roger.ri...@oracle.com>> wrote: >>> >>> Hi Lance, >>> >>> Can the name change be done using hg rename to preserve the continuity? >> >> I did do an hg rename, and just did it again in a different workspace: >> >> hg rename package.html package-info.java >> ljanders-mac:rowset ljanders$ hg status -mar >> M test/jdk/tools/jmod/hashes/HashesTest.java >> M test/jdk/tools/launcher/modules/addexports/AddExportsTest.java >> A src/java.sql.rowset/share/classes/javax/sql/rowset/package-info.java >> R src/java.sql.rowset/share/classes/javax/sql/rowset/package.html >>> >>> Also, while you are there, how about converting <code> to {@code...} etc. >> >> I do plan to do this, but thought I would keep things minimal for this >> updateand do that in a follow-on due to the renaming so it is easier to >> follow in the webrev. >> >> >> Best >> Lance >>> >>> Thanks, Roger >>> >>> >>> On 6/4/18 7:22 AM, Lance Andersen 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/%7Elancea/8201608/webrev.00/><http://cr.openjdk.java.net/~lancea/8201608/webrev.00/ >>>> <http://cr.openjdk.java.net/%7Elancea/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 >>>> >>>> Best >>>> Lance >>>> >>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif >>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>> >>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif >>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>> >>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif >>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>> >>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif >>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>>Lance Andersen| >>>> Principal Member of Technical Staff | +1.781.442.2037 >>>> Oracle Java Engineering >>>> 1 Network Drive >>>> Burlington, MA 01803 >>>> lance.ander...@oracle.com <mailto:lance.ander...@oracle.com> >>>> <mailto:lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>> >>>> >>>> >>> >> >> <oracle_sig_logo.gif> >> <http://oracle.com/us/design/oracle-email-sig-198324.gif> >> <http://oracle.com/us/design/oracle-email-sig-198324.gif> >> <http://oracle.com/us/design/oracle-email-sig-198324.gif> >> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| >> Principal Member of Technical Staff | +1.781.442.2037 >> Oracle Java Engineering >> 1 Network Drive >> Burlington, MA 01803 >> lance.ander...@oracle.com <mailto:lance.ander...@oracle.com> >> >> >> > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>