You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by cz...@apache.org on 2002/03/19 08:43:59 UTC
cvs commit: xml-cocoon2/src/java/org/apache/cocoon/components/pipeline AbstractEventPipeline.java
cziegeler 02/03/18 23:43:59
Modified: src/java/org/apache/cocoon/components/pipeline
AbstractEventPipeline.java
Log:
Improved performance a little bit and installed old test for SAXConnector
Revision Changes Path
1.7 +8 -6 xml-cocoon2/src/java/org/apache/cocoon/components/pipeline/AbstractEventPipeline.java
Index: AbstractEventPipeline.java
===================================================================
RCS file: /home/cvs/xml-cocoon2/src/java/org/apache/cocoon/components/pipeline/AbstractEventPipeline.java,v
retrieving revision 1.6
retrieving revision 1.7
diff -u -r1.6 -r1.7
--- AbstractEventPipeline.java 22 Feb 2002 07:00:11 -0000 1.6
+++ AbstractEventPipeline.java 19 Mar 2002 07:43:59 -0000 1.7
@@ -76,7 +76,7 @@
* @author <a href="mailto:sylvain@apache.org">Sylvain Wallez</a
* @author <a href="mailto:barozzi@nicolaken.com">Nicola Ken Barozzi</a>
* @author <a href="mailto:proyal@managingpartners.com">Peter Royal</a>
- * @version CVS $Id: AbstractEventPipeline.java,v 1.6 2002/02/22 07:00:11 cziegeler Exp $
+ * @version CVS $Id: AbstractEventPipeline.java,v 1.7 2002/03/19 07:43:59 cziegeler Exp $
*/
public abstract class AbstractEventPipeline
extends AbstractXMLProducer
@@ -99,12 +99,17 @@
/** the component manager set with compose() */
protected ComponentManager manager;
+ /** is a SAXConnector configured? */
+ protected boolean configuredSAXConnector;
+
/** the component manager set with compose() and recompose() */
protected ComponentManager newManager;
public void compose (ComponentManager manager)
throws ComponentException {
this.manager = manager;
+ // cache this test for a little bit performance
+ this.configuredSAXConnector = this.manager.hasComponent(SAXConnector.ROLE);
this.newManager = manager;
}
@@ -235,13 +240,10 @@
XMLProducer prev = (XMLProducer)this.generator;
XMLConsumer next;
- // Looked up in manager and not newManager (it's not a sitemap component)
- boolean configuredSAXConnector = this.manager.hasComponent(SAXConnector.ROLE);
-
try {
Iterator itt = this.transformers.iterator();
while ( itt.hasNext() ) {
- if (configuredSAXConnector) {
+ if (this.configuredSAXConnector) {
// connect SAXConnector
SAXConnector connect = (SAXConnector) this.manager.lookup(SAXConnector.ROLE);
connect.setup(environment,environment.getObjectModel(), null, null);
@@ -257,7 +259,7 @@
prev = trans;
}
- if (configuredSAXConnector) {
+ if (this.configuredSAXConnector) {
// insert SAXConnector
SAXConnector connect = (SAXConnector) this.manager.lookup(SAXConnector.ROLE);
connect.setup(environment,environment.getObjectModel(), null, null);
----------------------------------------------------------------------
In case of troubles, e-mail: webmaster@xml.apache.org
To unsubscribe, e-mail: cocoon-cvs-unsubscribe@xml.apache.org
For additional commands, e-mail: cocoon-cvs-help@xml.apache.org