Author: skitching
Date: Mon Jun 6 03:33:37 2005
New Revision: 180287
URL: http://svn.apache.org/viewcvs?rev=180287&view=rev
Log:
Fix bug introduced by recent changes. Thanks to Brian Stansberry for the patch.
Modified:
jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/impl/LogFactoryImpl.java
Modified:
jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/impl/LogFactoryImpl.java
URL:
http://svn.apache.org/viewcvs/jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/impl/LogFactoryImpl.java?rev=180287&r1=180286&r2=180287&view=diff
==============================================================================
---
jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/impl/LogFactoryImpl.java
(original)
+++
jakarta/commons/proper/logging/trunk/src/java/org/apache/commons/logging/impl/LogFactoryImpl.java
Mon Jun 6 03:33:37 2005
@@ -722,7 +722,16 @@
logAdapterClass = loadClass(logAdapterClassName);
constructor =
logAdapterClass.getConstructor(logConstructorSignature);
logAdapter = (Log) constructor.newInstance(params);
- } catch (NoClassDefFoundError e) {
+ } catch (ClassNotFoundException e) {
+ // We were unable to find the log adapter
+ String msg = "" + e.getMessage();
+ logDiagnostic(
+ "The log adapter "
+ + logAdapterClassName
+ + " is not available: "
+ + msg.trim());
+ return null;
+ } catch (NoClassDefFoundError e) {
// We were able to load the adapter but it had references to
// other classes that could not be found. This simply means that
// the underlying logger library could not be found.
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
