The followings are still in the webrev: - the unused AbstractTranslet: import java.util.Vector can be removed - comments in EmptySerializer and ToSAXHandler: SerializationHandler#setCdataSectionElements(java.util.Vector) The issue is SerializationHandler does not define a method "setCdataSectionElements" -Joe On 6/4/2014 2:04 PM, David Li wrote:
Hi Joe,
I have applied these changes and uploaded for review. http://cr.openjdk.java.net/~dli/8041523/webrev/. Let me know if you see anything else.
Thanks, David
On 6/4/2014 12:27 PM, huizhe wang wrote:
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