RFR (JDK 13/java.xml) 8222415: Xerces 2.12.0: Parsing Configuration
Joe Wang
huizhe.wang at oracle.com
Tue Apr 16 21:31:19 UTC 2019
Thanks Lance! Pushed after changing 3078/79 back to 3078.
Best,
Joe
On 4/16/19, 1:14 PM, Lance Andersen wrote:
> Hi Joe,
>
> Overall, I think the updates look OK. One minor correction/cleanup
> for before you push is in the XMLEntityManager line 3078/79 to move
> back the else to 3078
>
> No need to update the webrev
>
> HTH
>
> Best
> Lance
>> On Apr 15, 2019, at 3:46 PM, Joe Wang <huizhe.wang at oracle.com
>> <mailto:huizhe.wang at oracle.com>> wrote:
>>
>> Please review an update from Xerces in the configuration area. The
>> patch contains changes not easily measurable with test (e.g. a few
>> bytes for a parse, or small leaks for a file as large as a GB, or
>> none at all). But the change does improve in that a method (
>> readAndBuffer) that explicitly buffers was provided so that the
>> processes that required buffer all call it instead of the general
>> read method. While there's a small test added, the testing was
>> focused on getting the existing ones to pass.
>>
>> All XML-related tests in the core tests and JCK passed.
>>
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8222415
>> webrevs: http://cr.openjdk.java.net/~joehw/jdk13/8222415/webrev/
>> <http://cr.openjdk.java.net/%7Ejoehw/jdk13/8222415/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