You are viewing a plain text version of this content. The canonical link for it is here.
Posted to crimson-cvs@xml.apache.org by ed...@apache.org on 2002/04/23 02:09:23 UTC
cvs commit: xml-crimson/src/org/apache/crimson/parser XMLReaderImpl.java
edwingo 02/04/22 17:09:23
Modified: src/org/apache/crimson/parser XMLReaderImpl.java
Log:
Fix 2434 Crimson claims not to support features it does support
Revision Changes Path
1.4 +13 -7 xml-crimson/src/org/apache/crimson/parser/XMLReaderImpl.java
Index: XMLReaderImpl.java
===================================================================
RCS file: /home/cvs/xml-crimson/src/org/apache/crimson/parser/XMLReaderImpl.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -r1.3 -r1.4
--- XMLReaderImpl.java 1 Oct 2001 23:25:26 -0000 1.3
+++ XMLReaderImpl.java 23 Apr 2002 00:09:22 -0000 1.4
@@ -1,5 +1,5 @@
/*
- * $Id: XMLReaderImpl.java,v 1.3 2001/10/01 23:25:26 edwingo Exp $
+ * $Id: XMLReaderImpl.java,v 1.4 2002/04/23 00:09:22 edwingo Exp $
*
* The Apache Software License, Version 1.1
*
@@ -71,7 +71,7 @@
* This implements the SAX2 XMLReader.
* @author Rajiv Mordani
* @author Edwin Goei
- * @version $Revision: 1.3 $
+ * @version $Revision: 1.4 $
*/
public class XMLReaderImpl implements XMLReader {
@@ -199,16 +199,22 @@
parser = null;
}
validation = state;
- } else if (name.equals(STRING_INTERNING)) {
+ } else if (name.equals(STRING_INTERNING) ||
+ name.equals(EXTERNAL_GENERAL) ||
+ name.equals(EXTERNAL_PARAMETER)) {
+ checkNotParsing("feature", name);
if (state == false) {
throw new SAXNotSupportedException("Feature: " + name
+ " State: false");
}
// else true is OK
- } else if (name.equals(EXTERNAL_GENERAL) ||
- name.equals(EXTERNAL_PARAMETER) ||
- name.equals(LEXICAL_PARAMETER_ENTITIES)) {
- throw new SAXNotSupportedException("Feature: " + name);
+ } else if (name.equals(LEXICAL_PARAMETER_ENTITIES)) {
+ checkNotParsing("feature", name);
+ if (state == true) {
+ throw new SAXNotSupportedException("Feature: " + name
+ + " State: true");
+ }
+ // else false is OK
} else {
throw new SAXNotRecognizedException("Feature: " + name);
}
---------------------------------------------------------------------
To unsubscribe, e-mail: crimson-cvs-unsubscribe@xml.apache.org
For additional commands, e-mail: crimson-cvs-help@xml.apache.org