RFR (JDK 13/java.xml) 8222415: Xerces 2.12.0: Parsing Configuration
Lance Andersen
lance.andersen at oracle.com
Tue Apr 16 20:14:23 UTC 2019
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> 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/
>
> 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