You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by js...@apache.org on 2002/12/16 11:51:06 UTC
cvs commit: jakarta-commons-sandbox/jelly/src/java/org/apache/commons/jelly/parser XMLParser.java
jstrachan 2002/12/16 02:51:06
Modified: jelly project.xml
jelly/src/java/org/apache/commons/jelly/parser
XMLParser.java
Log:
Applied patch supplied by Todd Jonker to improve the error reporting when a Tag library cannot be loaded and fixed a typeo in the exception message
Revision Changes Path
1.101 +8 -0 jakarta-commons-sandbox/jelly/project.xml
Index: project.xml
===================================================================
RCS file: /home/cvs/jakarta-commons-sandbox/jelly/project.xml,v
retrieving revision 1.100
retrieving revision 1.101
diff -u -r1.100 -r1.101
--- project.xml 11 Dec 2002 20:02:16 -0000 1.100
+++ project.xml 16 Dec 2002 10:51:06 -0000 1.101
@@ -215,6 +215,14 @@
<role>Developer</role>
</roles>
</contributor>
+ <contributor>
+ <name>Todd Jonker</name>
+ <email></email>
+ <organization></organization>
+ <roles>
+ <role>Developer</role>
+ </roles>
+ </contributor>
</contributors>
<dependencies>
1.41 +15 -6 jakarta-commons-sandbox/jelly/src/java/org/apache/commons/jelly/parser/XMLParser.java
Index: XMLParser.java
===================================================================
RCS file: /home/cvs/jakarta-commons-sandbox/jelly/src/java/org/apache/commons/jelly/parser/XMLParser.java,v
retrieving revision 1.40
retrieving revision 1.41
diff -u -r1.40 -r1.41
--- XMLParser.java 10 Nov 2002 11:02:48 -0000 1.40
+++ XMLParser.java 16 Dec 2002 10:51:06 -0000 1.41
@@ -1033,6 +1033,15 @@
catch (ClassNotFoundException e) {
log.warn("Could not load class: " + uri + " so disabling the taglib");
}
+ catch (IllegalAccessException e) {
+ log.warn("Constructor for class is not accessible: " + uri + " so disabling the taglib");
+ }
+ catch (InstantiationException e) {
+ log.warn("Class could not be instantiated: " + uri + " so disabling the taglib");
+ }
+ catch (ClassCastException e) {
+ log.warn("Class is not a TagLibrary: " + uri + " so disabling the taglib");
+ }
}
}
if (taglib != null) {
@@ -1201,7 +1210,7 @@
+ locator.getLineNumber()
+ ", "
+ locator.getColumnNumber()
- + ": "
+ + "): "
+ message;
if (e != null) {
return new SAXParseException(error, locator, e);
--
To unsubscribe, e-mail: <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>