cvs commit: xml-cocoon2/src/org/apache/cocoon/components/parser JaxpParser.java

[cziegeler]

cziegeler    01/10/31 03:37:23

  Modified:    src/org/apache/cocoon/components/parser JaxpParser.java
  Log:
  Minor performance update. Used classes are only logged during configuration
  
  Revision  Changes    Path
  1.12      +12 -8     
xml-cocoon2/src/org/apache/cocoon/components/parser/JaxpParser.java
  
  Index: JaxpParser.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/org/apache/cocoon/components/parser/JaxpParser.java,v
  retrieving revision 1.11
  retrieving revision 1.12
  diff -u -r1.11 -r1.12
  --- JaxpParser.java   2001/10/29 12:22:01     1.11
  +++ JaxpParser.java   2001/10/31 11:37:23     1.12
  @@ -31,7 +31,7 @@
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Berin Loritsch</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Carsten Ziegeler</a>
  - * @version CVS $Revision: 1.11 $ $Date: 2001/10/29 12:22:01 $
  + * @version CVS $Revision: 1.12 $ $Date: 2001/10/31 11:37:23 $
    */
   public class JaxpParser extends AbstractXMLProducer
   implements Parser, ErrorHandler, Composable, Configurable, Recyclable {
  @@ -76,21 +76,26 @@
           this.factory.setValidating(validate);
           this.docfactory.setNamespaceAware(true);
           this.docfactory.setValidating(validate);
  +        getLogger().debug("SAXParserFactory: " + 
ClassUtils.which(this.factory.getClass()));
  +        getLogger().debug("DocumentBuilderFactory: " + 
ClassUtils.which(this.docfactory.getClass()));
  +        try {
  +            SAXParser parser = this.factory.newSAXParser();
  +            getLogger().debug("SAXParser: " + ClassUtils.which(parser.getClass()));
  +        } catch (Exception e) {
  +            getLogger().error("Cannot produce a valid parser", e);
  +            // we ignore it here for now
  +        }
       }
   
       public void parse(InputSource in)
       throws SAXException, IOException {
           SAXParser parser = null;
   
  -        getLogger().debug("SAXParserFactory: " + 
ClassUtils.which(this.factory.getClass()));
  -        getLogger().debug("DocumentBuilderFactory: " + 
ClassUtils.which(this.docfactory.getClass()));
  -
           try {
               parser = this.factory.newSAXParser();
  -            getLogger().debug("SAXParser: " + ClassUtils.which(parser.getClass()));
  -        } catch (Exception e) {
  +        } catch (ParserConfigurationException e) {
               getLogger().error("Cannot produce a valid parser", e);
  -            throw new SAXException("Could not get valid parser" + e.getMessage());
  +            throw new SAXException("Could not get valid parser" + e.getMessage(), 
e);
           }
   
           XMLReader reader = parser.getXMLReader();
  @@ -119,7 +124,6 @@
   
           try {
               builder = this.docfactory.newDocumentBuilder();
  -            getLogger().debug("DocumentBuilder: " + 
ClassUtils.which(builder.getClass()));
           } catch (ParserConfigurationException pce) {
               getLogger().error("Could not create DocumentBuilder", pce);
               return null;
  
  
  

----------------------------------------------------------------------
In case of troubles, e-mail:     [EMAIL PROTECTED]
To unsubscribe, e-mail:          [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]