Thanks Lance!
-Joe
On 11/21/19 3:59 PM, Lance Andersen wrote:
Hi Joe,
Looks OK overall.
Best
Lance
On Nov 20, 2019, at 4:48 PM, Joe Wang <huizhe.w...@oracle.com
<mailto:huizhe.w...@oracle.com>> wrote:
Hi,
Please review an update to Java CUP. This is an effort to move to the
latest version, v0.11b. JCUP is used by Xalan to generate
XPathParser. Included in the JDK are runtime classes and XPathParser.
In CUP 0.11b, the main additions to the runtime were SymbolFactory
and ComplexSymbol that were used to report a bit more details
(locations) about the parsing process. They were not used by Xalan
nor the JDK since the benefit of these additions were in error
reporting, but Xalan and JDK used their own reporting mechanism. I
could have removed the JCUP error report process altogether, but
decided to keep it in case it might be used for debugging.
The main change for this update therefore is to the XPathParser
itself. Since JCUP has been stable, and no major Xalan update in
sight, I took the liberty to re-format the whole class, plus some
cleanup (mainly boxing/unboxing), that constitute the changes before
the inner class parser_actions. Within parser_actions, the main
change is that the 0.11b fixed the order of the switch block. Other
than that, I removed many unused variables, and also formatted the
long lines.
There were no other material changes. So there's no new test, all
existing tests (including JCK) passed.
JBS: https://bugs.openjdk.java.net/browse/JDK-8233548
webrevs: http://cr.openjdk.java.net/~joehw/jdk14/8233548/webrev/
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>