DO NOT REPLY [Bug 23113] - org.apache.xml.serializer.Serializer.reset() resets ampersand entity handling

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=23113>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=23113

org.apache.xml.serializer.Serializer.reset() resets ampersand entity handling

minchau@ca.ibm.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |xalan-dev@xml.apache.org
         AssignedTo|xalan-dev@xml.apache.org    |minchau@ca.ibm.com



------- Additional Comments From minchau@ca.ibm.com  2003-09-11 21:33 -------
The problem is with the ToStream.resetToStream() method.
It incorrectly sets the field m_escaping to false, when the original value
that the field has when a ToStream object is created is true.  This is a one 
line fix.

This exposed a second problem where the same resetToStream() method set 
m_charInfo to null. Admittedly there was a comment with ?? on that line.
Also a one line fix NOT to set m_charInfo to null.
------------------------------------
A patch will be attached to the bug shortly.

Please be aware that there is a restriction, and this was true before the 
patch.  If you create a serializer, say to serializer to XML and you want to re-
use the serializer, it will stay an XML serializer.  Changing the properties via
s.setOutputFormat(op); can never change the type of serializer (XML, TEXT, 
HTML).

Regards, 
Brian Minchau