RFR [14/java.xml] 8233548: Update CUP to v0.11b

Joe Wang huizhe.wang at oracle.com
Fri Nov 22 00:26:35 UTC 2019


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.wang at oracle.com 
>> <mailto:huizhe.wang at 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.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
>
>
>



More information about the core-libs-dev mailing list