RFR (JAXP): 8035437: Xerces Update: xml/serialize/DOMSerializerImpl
Aleksej Efimov
aleksej.efimov at oracle.com
Mon Mar 31 15:09:41 UTC 2014
Lance, thank you for the review.
Aleksej
On 03/28/2014 03:33 PM, Lance @ Oracle wrote:
> With the change to stringbuilder I am ok with it
>
> Best
> Lance
>
>
> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance
> Andersen| Principal Member of Technical Staff | +1.781.442.2037
> <tel:+1.781.442.2037>
> Oracle Java Engineering
> 1 Network Drive <x-apple-data-detectors://34/0>
> Burlington, MA 01803 <x-apple-data-detectors://34/0>
> Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
> Sent from my iPad
>
> On Mar 26, 2014, at 1:44 PM, Aleksej Efimov <aleksej.efimov at oracle.com
> <mailto:aleksej.efimov at oracle.com>> wrote:
>
>> Lance,
>> The document implementation (DocumentImpl) returned by
>> 'builder.newDocument()' should implement org.w3c.dom.Node and
>> org.w3c.dom.Document interfaces, but shouldn't have the
>> 'getXmlVersion' method (in Node, Document and DocumentImpl) - it's a
>> main item required by Apache bug. So it means, that we need to
>> substitute all three classes (Document,Node and DocumentImpl). The
>> test should be executed with JDKs Node, Document, but DocumentImpl
>> should be compiled against incorrect interfaces. I don't see, how I
>> can implement it with jtreg keywords (the main problem is splitting
>> the compilation of DocumentImpl (fake Node, Document) and the test
>> run (only fake DocumentImpl).
>>
>> The only reason why StringBuffer is not used here - the process of
>> how updates were moved from Xerces to JAXP: revision by revision. And
>> I don't think that it should prevent us from replacing it with faster
>> StringBuilder =). Will change it to StringBuilder.
>>
>> Thank you,
>> Aleksej
>>
>> On 03/26/2014 09:04 PM, Lance Andersen wrote:
>>> Any reason not to use StringBuilder in place of StringBuffer
>>> in XMLEntityManager.java
>>>
>>> other than the above and my question on the need of using run.sh, it
>>> is OK
>>>
>>> On Mar 26, 2014, at 12:54 PM, Lance Andersen
>>> <lance.andersen at oracle.com <mailto:lance.andersen at oracle.com>> wrote:
>>>
>>>> Hi Aleksej,
>>>>
>>>> Do you really need to use run.sh? If possible it would be better
>>>> to try and avoid providing a script and use the jtreg keywords.
>>>>
>>>> Best
>>>> Lance
>>>> On Mar 26, 2014, at 12:39 PM, Aleksej Efimov
>>>> <aleksej.efimov at oracle.com <mailto:aleksej.efimov at oracle.com>> wrote:
>>>>
>>>>> Hello,
>>>>>
>>>>> Can I ask for a review for the update of DOMSerializerImpl class
>>>>> [1] to the latest Apache Xerces implementation.
>>>>> The following changes were made:
>>>>> 1. The DOMSerializerImpl update: The Xerces class was used as a
>>>>> base [2]; the 476047 and 473125 revisions from Xerces was
>>>>> excluded (they will be resolved as a part of JDK-8035467 - move to
>>>>> Xalan serializer bug); the jaxp 632 revision was applied to new
>>>>> version of this class.
>>>>> 2. The DOMSerializerImpl.java file was reformated, because of
>>>>> not-good formatting in original file from Apache repo [2].
>>>>> 3. Other changes related to the DOMSerializerImpl update were
>>>>> applied from Xerces repo to JDK(the full revisions list can be
>>>>> found in the JBS [1]).
>>>>> 4. All license headers were updated to the latest version of
>>>>> Apache header in modified files.
>>>>> 5. New regression test was added to reproduce a problem described
>>>>> in Apache bug XERCESJ-1007 [3].
>>>>>
>>>>> Webrevs for these changes:
>>>>> http://cr.openjdk.java.net/~aefimov/8035437/webrev.00/jaxp
>>>>> <http://cr.openjdk.java.net/%7Eaefimov/8035437/webrev.00/jaxp>
>>>>> http://cr.openjdk.java.net/~aefimov/8035437/webrev.00/jdk
>>>>> <http://cr.openjdk.java.net/%7Eaefimov/8035437/webrev.00/jdk>
>>>>>
>>>>> Testing:
>>>>> JCK tests (api/xsl api/xinclude api/javax_xml api/org_xml
>>>>> xml_schema): pass
>>>>> JTREG tests (javax/xml, jdk_other): pass
>>>>>
>>>>> Thank you,
>>>>> Aleksej
>>>>>
>>>>> [1] https://bugs.openjdk.java.net/browse/JDK-6339023
>>>>> [2]
>>>>> http://svn.apache.org/viewvc/xerces/java/trunk/src/org/apache/xml/serialize/DOMSerializerImpl.java?revision=944789&view=markup
>>>>> [3] https://issues.apache.org/jira/browse/XERCESJ-1007
>>>>>
>>>>
>>>>
>>>>
>>>> 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>
>>>>
>>>>
>>>>
>>>
>>> <mime-attachment.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>
>>> <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