elena       2003/11/18 15:08:03

  Modified:    java/src/org/apache/xerces/parsers AbstractSAXParser.java
               java/src/org/apache/xerces/util ErrorHandlerWrapper.java
  Log:
  Upgrading to SAX 2.0.1. support
  
  Revision  Changes    Path
  1.44      +2 -30     
xml-xerces/java/src/org/apache/xerces/parsers/AbstractSAXParser.java
  
  Index: AbstractSAXParser.java
  ===================================================================
  RCS file: 
/home/cvs/xml-xerces/java/src/org/apache/xerces/parsers/AbstractSAXParser.java,v
  retrieving revision 1.43
  retrieving revision 1.44
  diff -u -r1.43 -r1.44
  --- AbstractSAXParser.java    11 Nov 2003 20:15:01 -0000      1.43
  +++ AbstractSAXParser.java    18 Nov 2003 23:08:03 -0000      1.44
  @@ -1227,10 +1227,6 @@
        */
       public void setEntityResolver(EntityResolver resolver) {
   
  -        // not per SAX 1, but per SAX 2.0 and JAXP 1.1 expectations
  -        if(resolver == null) {
  -            throw new NullPointerException();
  -        }
           try {
               fConfiguration.setProperty(ENTITY_RESOLVER,
                                          new EntityResolverWrapper(resolver));
  @@ -1280,16 +1276,10 @@
        * handler immediately.</p>
        *
        * @param errorHandler The error handler.
  -     * @exception java.lang.NullPointerException If the handler
  -     *            argument is null.
        * @see #getErrorHandler
        */
       public void setErrorHandler(ErrorHandler errorHandler) {
   
  -        // not per SAX 1, but per SAX 2.0 and JAXP 1.1 expectations
  -        if(errorHandler == null) {
  -            throw new NullPointerException();
  -        }
           try {
               fConfiguration.setProperty(ERROR_HANDLER,
                                          new ErrorHandlerWrapper(errorHandler));
  @@ -1354,21 +1344,10 @@
        *
        * @param dtdHandler The DTD handler.
        *
  -     * @exception java.lang.NullPointerException If the handler
  -     *            argument is null.
  -     *
  +
        * @see #getDTDHandler
        */
       public void setDTDHandler(DTDHandler dtdHandler) {
  -        // REVISIT: SAX1 doesn't require a null pointer exception
  -        //          to be thrown but SAX2 does. [Q] How do we
  -        //          resolve this? Currently I'm erring on the side
  -        //          of SAX2. -Ac
  -        // This is what the JCK requires; 2.0 is assumed to supersede the
  -        // SAX 1 behaviour - neilg
  -        if (dtdHandler == null) {
  -            throw new NullPointerException();
  -        }
           fDTDHandler = dtdHandler;
       } // setDTDHandler(DTDHandler)
   
  @@ -1411,16 +1390,9 @@
        *
        * @param contentHandler The content handler.
        *
  -     * @exception java.lang.NullPointerException If the handler
  -     *            argument is null.
  -     *
        * @see #getContentHandler
        */
       public void setContentHandler(ContentHandler contentHandler) {
  -        // not per SAX 1, but per SAX 2.0 and JAXP 1.1 expectations
  -        if(contentHandler == null) {
  -            throw new NullPointerException();
  -        }
           fContentHandler = contentHandler;
       } // setContentHandler(ContentHandler)
   
  
  
  
  1.9       +40 -31    
xml-xerces/java/src/org/apache/xerces/util/ErrorHandlerWrapper.java
  
  Index: ErrorHandlerWrapper.java
  ===================================================================
  RCS file: 
/home/cvs/xml-xerces/java/src/org/apache/xerces/util/ErrorHandlerWrapper.java,v
  retrieving revision 1.8
  retrieving revision 1.9
  diff -u -r1.8 -r1.9
  --- ErrorHandlerWrapper.java  8 May 2003 20:11:59 -0000       1.8
  +++ ErrorHandlerWrapper.java  18 Nov 2003 23:08:03 -0000      1.9
  @@ -133,17 +133,20 @@
       public void warning(String domain, String key, 
                           XMLParseException exception) throws XNIException {
   
  -        SAXParseException saxException = createSAXParseException(exception);
  -        try {
  -            fErrorHandler.warning(saxException);
  +        if (fErrorHandler != null) {
  +             SAXParseException saxException = createSAXParseException(exception);
  +             
  +             try {
  +                     fErrorHandler.warning(saxException);
  +             }
  +             catch (SAXParseException e) {
  +                     throw createXMLParseException(e);
  +             }
  +             catch (SAXException e) {
  +                     throw createXNIException(e);
  +             }
           }
  -        catch (SAXParseException e) {
  -            throw createXMLParseException(e);
  -        }
  -        catch (SAXException e) {
  -            throw createXNIException(e);
  -        }
  -
  +        
       } // warning(String,String,XMLParseException)
   
       /**
  @@ -164,16 +167,19 @@
        */
       public void error(String domain, String key, 
                         XMLParseException exception) throws XNIException {
  -
  -        SAXParseException saxException = createSAXParseException(exception);
  -        try {
  -            fErrorHandler.error(saxException);
  -        }
  -        catch (SAXParseException e) {
  -            throw createXMLParseException(e);
  -        }
  -        catch (SAXException e) {
  -            throw createXNIException(e);
  +        
  +        if (fErrorHandler != null) {
  +             SAXParseException saxException = createSAXParseException(exception);
  +             
  +             try {
  +                     fErrorHandler.error(saxException);
  +             }
  +             catch (SAXParseException e) {
  +                     throw createXMLParseException(e);
  +             }
  +             catch (SAXException e) {
  +                     throw createXNIException(e);
  +             }
           }
   
       } // error(String,String,XMLParseException)
  @@ -204,16 +210,19 @@
        */
       public void fatalError(String domain, String key, 
                              XMLParseException exception) throws XNIException {
  -
  -        SAXParseException saxException = createSAXParseException(exception);
  -        try {
  -            fErrorHandler.fatalError(saxException);
  -        }
  -        catch (SAXParseException e) {
  -            throw createXMLParseException(e);
  -        }
  -        catch (SAXException e) {
  -            throw createXNIException(e);
  +                             
  +        if (fErrorHandler != null) {
  +             SAXParseException saxException = createSAXParseException(exception);
  +             
  +             try {
  +                     fErrorHandler.fatalError(saxException);
  +             }
  +             catch (SAXParseException e) {
  +                     throw createXMLParseException(e);
  +             }
  +             catch (SAXException e) {
  +                     throw createXNIException(e);
  +             }
           }
   
       } // fatalError(String,String,XMLParseException)
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to