You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@xalan.apache.org by jk...@apache.org on 2001/01/09 20:41:48 UTC
cvs commit: xml-xalan/java/src/org/apache/xalan/stree SourceTreeHandler.java
jkesselm 01/01/09 11:41:47
Modified: java/src/org/apache/xalan/stree SourceTreeHandler.java
Log:
Close a small timing exposure in endElement().
Revision Changes Path
1.29 +7 -2 xml-xalan/java/src/org/apache/xalan/stree/SourceTreeHandler.java
Index: SourceTreeHandler.java
===================================================================
RCS file: /home/cvs/xml-xalan/java/src/org/apache/xalan/stree/SourceTreeHandler.java,v
retrieving revision 1.28
retrieving revision 1.29
diff -u -r1.28 -r1.29
--- SourceTreeHandler.java 2001/01/02 03:36:43 1.28
+++ SourceTreeHandler.java 2001/01/09 19:41:43 1.29
@@ -545,17 +545,22 @@
synchronized (m_root)
{
- ((Parent) m_sourceTreeHandler.getCurrentNode()).setComplete(true);
+ Parent myElement=(Parent)(m_sourceTreeHandler.getCurrentNode());
m_sourceTreeHandler.endElement(ns, localName, name);
+ // Mark as complete only after endElement has had a chance to flush
+ // any pending work (Text nodes in particular)
+ myElement.setComplete(true);
+
m_shouldStripWS = m_shouldStripWhitespaceStack.popAndTop();
+
}
notifyWaiters();
}
- /** Flaf indicating whether we got a CDATA event */
+ /** Flag indicating whether we got a CDATA event */
private boolean m_isCData = false;
/**