You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by ri...@apache.org on 2009/01/31 01:52:25 UTC
svn commit: r739465 -
/felix/trunk/framework/src/main/java/org/apache/felix/framework/searchpolicy/ModuleImpl.java
Author: rickhall
Date: Sat Jan 31 00:52:25 2009
New Revision: 739465
URL: http://svn.apache.org/viewvc?rev=739465&view=rev
Log:
More [minor] refactoring. (FELIX-851)
Modified:
felix/trunk/framework/src/main/java/org/apache/felix/framework/searchpolicy/ModuleImpl.java
Modified: felix/trunk/framework/src/main/java/org/apache/felix/framework/searchpolicy/ModuleImpl.java
URL: http://svn.apache.org/viewvc/felix/trunk/framework/src/main/java/org/apache/felix/framework/searchpolicy/ModuleImpl.java?rev=739465&r1=739464&r2=739465&view=diff
==============================================================================
--- felix/trunk/framework/src/main/java/org/apache/felix/framework/searchpolicy/ModuleImpl.java (original)
+++ felix/trunk/framework/src/main/java/org/apache/felix/framework/searchpolicy/ModuleImpl.java Sat Jan 31 00:52:25 2009
@@ -441,15 +441,6 @@
{
return getClassLoader().loadClass(name);
}
- catch (ClassNotFoundException ex)
- {
-// TODO: REFACTOR - Should this log?
- m_logger.log(
- Logger.LOG_WARNING,
- ex.getMessage(),
- ex);
- throw ex;
- }
finally
{
pkgCycleSet.remove(name);
@@ -475,10 +466,14 @@
}
catch (ClassNotFoundException ex)
{
+ // This should never be thrown because we are loading resources.
}
catch (ResourceNotFoundException ex)
{
-// TODO: REFACTOR - Should this log?
+ m_logger.log(
+ Logger.LOG_DEBUG,
+ ex.getMessage(),
+ ex);
}
finally
{