You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@xerces.apache.org by el...@apache.org on 2003/11/19 00:08:03 UTC
cvs commit: xml-xerces/java/src/org/apache/xerces/util ErrorHandlerWrapper.java
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: xerces-cvs-unsubscribe@xml.apache.org
For additional commands, e-mail: xerces-cvs-help@xml.apache.org