You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@xalan.apache.org by sb...@apache.org on 2001/01/07 05:19:17 UTC
cvs commit: xml-xalan/java/src/org/apache/xalan/transformer QueuedSAXEvent.java QueuedStartDocument.java QueuedStartElement.java
sboag 01/01/06 20:19:17
Modified: java/src/org/apache/xalan/transformer QueuedSAXEvent.java
QueuedStartDocument.java QueuedStartElement.java
Log:
Don't create GenerateEvents if there are no tracers. These event
objects, which weren't being used, were showing up as 11% of
garbage collected objects in JProbe, for large-all_well.xml.
Revision Changes Path
1.6 +3 -3 xml-xalan/java/src/org/apache/xalan/transformer/QueuedSAXEvent.java
Index: QueuedSAXEvent.java
===================================================================
RCS file: /home/cvs/xml-xalan/java/src/org/apache/xalan/transformer/QueuedSAXEvent.java,v
retrieving revision 1.5
retrieving revision 1.6
diff -u -r1.5 -r1.6
--- QueuedSAXEvent.java 2000/12/13 17:22:58 1.5
+++ QueuedSAXEvent.java 2001/01/07 04:19:17 1.6
@@ -147,11 +147,11 @@
*/
protected void fireGenerateEvent(int type, String name, Attributes attrs)
{
-
- GenerateEvent ge = new GenerateEvent(m_transformer, type, name, attrs);
-
if (null != m_traceManager)
+ {
+ GenerateEvent ge = new GenerateEvent(m_transformer, type, name, attrs);
m_traceManager.fireGenerateEvent(ge);
+ }
}
/**
1.6 +5 -1 xml-xalan/java/src/org/apache/xalan/transformer/QueuedStartDocument.java
Index: QueuedStartDocument.java
===================================================================
RCS file: /home/cvs/xml-xalan/java/src/org/apache/xalan/transformer/QueuedStartDocument.java,v
retrieving revision 1.5
retrieving revision 1.6
diff -u -r1.5 -r1.6
--- QueuedStartDocument.java 2000/12/13 17:22:59 1.5
+++ QueuedStartDocument.java 2001/01/07 04:19:17 1.6
@@ -106,7 +106,11 @@
if (isPending)
{
m_contentHandler.startDocument();
- fireGenerateEvent(GenerateEvent.EVENTTYPE_STARTDOCUMENT, null, null);
+
+ if(null != m_traceManager)
+ {
+ fireGenerateEvent(GenerateEvent.EVENTTYPE_STARTDOCUMENT, null, null);
+ }
ContentHandler chandler = getContentHandler();
1.7 +5 -2 xml-xalan/java/src/org/apache/xalan/transformer/QueuedStartElement.java
Index: QueuedStartElement.java
===================================================================
RCS file: /home/cvs/xml-xalan/java/src/org/apache/xalan/transformer/QueuedStartElement.java,v
retrieving revision 1.6
retrieving revision 1.7
diff -u -r1.6 -r1.7
--- QueuedStartElement.java 2000/12/13 17:23:00 1.6
+++ QueuedStartElement.java 2001/01/07 04:19:17 1.7
@@ -285,8 +285,11 @@
{
m_contentHandler.startElement(m_url, m_localName, m_name,
m_attributes);
- fireGenerateEvent(GenerateEvent.EVENTTYPE_STARTELEMENT, m_name,
- m_attributes);
+ if(null != m_traceManager)
+ {
+ fireGenerateEvent(GenerateEvent.EVENTTYPE_STARTELEMENT, m_name,
+ m_attributes);
+ }
}
reset();