Hi Joe, Overall this looks good and also cleans up a couple of typos :-)
One nit in both package-info @apiNote, you will notice an extra space before the was which could be removed before you push Best Lance > On Sep 19, 2019, at 8:00 PM, Joe Wang <huizhe.w...@oracle.com> wrote: > > Please review a follow-up doc clarification patch after 8230814 [1]. In this > patch, the statement with a reference to the SAX project is moved to an > apiNote in package/sub-package description to reflect the fact that it is a > historical note in nature. The license related text that appears in the class > description of every SAX class is removed and consolidated with the existing > text in the license header. > > JBS: https://bugs.openjdk.java.net/browse/JDK-8231083 > webrev: http://cr.openjdk.java.net/~joehw/jdk14/8231083/webrev/index.html > > > [1] https://bugs.openjdk.java.net/browse/JDK-8230814 > > Thanks, > Joe <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>