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;
     }