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/09 23:02:22 UTC
svn commit: r733167 -
/felix/trunk/framework/src/main/java/org/apache/felix/framework/searchpolicy/ModuleImpl.java
Author: rickhall
Date: Fri Jan 9 14:02:22 2009
New Revision: 733167
URL: http://svn.apache.org/viewvc?rev=733167&view=rev
Log:
Minor changes. (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=733167&r1=733166&r2=733167&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 Fri Jan 9 14:02:22 2009
@@ -621,10 +621,7 @@
{
m_fragmentContents[i].close();
}
- synchronized (this)
- {
- m_classLoader = null;
- }
+ m_classLoader = null;
}
public IContent getContent()
@@ -819,15 +816,12 @@
return getClassPath()[index - 1].getEntryAsStream(urlPath);
}
- private ModuleClassLoader getClassLoader()
+ private synchronized ModuleClassLoader getClassLoader()
{
- synchronized (this)
+ if (m_classLoader == null)
{
- if (m_classLoader == null)
- {
- m_classLoader = m_secureAction.createModuleClassLoader(
- this, m_protectionDomain);
- }
+ m_classLoader = m_secureAction.createModuleClassLoader(
+ this, m_protectionDomain);
}
return m_classLoader;
}