You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by sk...@apache.org on 2005/02/24 05:09:45 UTC
svn commit: r155162 -
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/plugins/Declaration.java
Author: skitching
Date: Wed Feb 23 20:09:43 2005
New Revision: 155162
URL: http://svn.apache.org/viewcvs?view=rev&rev=155162
Log:
Handle ruleLoader==null, which is possible.
Modified:
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/plugins/Declaration.java
Modified: jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/plugins/Declaration.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/plugins/Declaration.java?view=diff&r1=155161&r2=155162
==============================================================================
--- jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/plugins/Declaration.java (original)
+++ jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/plugins/Declaration.java Wed Feb 23 20:09:43 2005
@@ -232,6 +232,13 @@
log.debug("configure being called!");
}
- ruleLoader.addRules(context);
+ if (ruleLoader != null) {
+ // The ruleloader can be null if findLoader returned null, ie
+ // none of the registered RuleFinder objects could locate a
+ // source of custom rules for the specified class. This
+ // probably indicates that something is wrong: maybe we should
+ // log a warning here?
+ ruleLoader.addRules(context);
+ }
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org