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:[EMAIL PROTECTED]";>Sylvain Wallez</a
    * @author <a href="mailto:[EMAIL PROTECTED]";>Nicola Ken Barozzi</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>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:     [EMAIL PROTECTED]
To unsubscribe, e-mail:          [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to