You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-dev@axis.apache.org by di...@apache.org on 2002/09/05 19:31:03 UTC
cvs commit: xml-axis/java/src/org/apache/axis/configuration EngineConfigurationFactoryFinder.java
dims 2002/09/05 10:31:03
Modified: java/src/org/apache/axis/configuration
EngineConfigurationFactoryFinder.java
Log:
Call log.debug ONLY if log.isDebugEnabled is true. This seems to fix "NPE with multithreaded client in RC1" problem reported by "Volanis, Alexander" <AV...@rsasecurity.com>
Details:
- http://marc.theaimsgroup.com/?t=103124516800009&r=1&w=2
- http://marc.theaimsgroup.com/?t=103120473600002&r=1&w=2
Revision Changes Path
1.7 +2 -1 xml-axis/java/src/org/apache/axis/configuration/EngineConfigurationFactoryFinder.java
Index: EngineConfigurationFactoryFinder.java
===================================================================
RCS file: /home/cvs/xml-axis/java/src/org/apache/axis/configuration/EngineConfigurationFactoryFinder.java,v
retrieving revision 1.6
retrieving revision 1.7
diff -u -r1.6 -r1.7
--- EngineConfigurationFactoryFinder.java 30 Aug 2002 02:31:18 -0000 1.6
+++ EngineConfigurationFactoryFinder.java 5 Sep 2002 17:31:03 -0000 1.7
@@ -176,7 +176,8 @@
}
if (factory != null) {
- log.debug(JavaUtils.getMessage("engineFactory", factory.getClass().getName()));
+ if(log.isDebugEnabled())
+ log.debug(JavaUtils.getMessage("engineFactory", factory.getClass().getName()));
} else {
log.error(JavaUtils.getMessage("engineConfigFactoryMissing"));
// we should be throwing an exception here,