Hi David,
Good work!
This is a major patch in Xalan, one last in Xalan 2.7.1. Previously, it
caused many failures in JCK and SQE regression tests. I see that you've
resolved those failures with the fix stated below, and getting all tests
to pass.
Some minor cleanup may be desirable, such as:
- the unused AbstractTranslet: import java.util.Vector
- comments in EmptySerializer and ToSAXHandler:
SerializationHandler#setCdataSectionElements(java.util.Vector)
- comment in the new method initCdataElems in SerializerBase that "we
just ended a URI, add the URI to the vector" although it didn't.
- comments in ToStream about setCdataSectionElements(Vector v)
Thanks,
Joe
On 6/4/2014 11:35 AM, David Li wrote:
Hi,
This is an update to JAXP serialization from Apache Xalan. For
details, please refer to:
https://bugs.openjdk.java.net/browse/JDK-8041523.
Webrevs: http://cr.openjdk.java.net/~dli/8041523/webrev/
Summary of changes.
* Changes from the Apache bug referenced in JDK-8041523
* Update ToStream.setCdataSectionElements to a newer version from the
most recent Apache ToStream.java source file. This was causing the
JCK test failures noted in the bug. Note, there are two
setCdataSectionElement methods in ToStream.java. The one with the bug
has been fixed.
* Some updates to accommodate ArrayList, HashMap, and Generics.
* Bundled in changes for JDK-8037259. This was an xpointer bug.
Existing tests: JAXP SQE and unit tests passed. JCK tests passed.
Thanks,
David