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/02/03 17:41:58 UTC

svn commit: r740346 - /felix/trunk/framework/src/main/java/org/apache/felix/framework/BundleImpl.java

Author: rickhall
Date: Tue Feb  3 16:41:58 2009
New Revision: 740346

URL: http://svn.apache.org/viewvc?rev=740346&view=rev
Log:
Made access to BundleImpl.m_state volatile rather than synchronized. (FELIX-908)

Modified:
    felix/trunk/framework/src/main/java/org/apache/felix/framework/BundleImpl.java

Modified: felix/trunk/framework/src/main/java/org/apache/felix/framework/BundleImpl.java
URL: http://svn.apache.org/viewvc/felix/trunk/framework/src/main/java/org/apache/felix/framework/BundleImpl.java?rev=740346&r1=740345&r2=740346&view=diff
==============================================================================
--- felix/trunk/framework/src/main/java/org/apache/felix/framework/BundleImpl.java (original)
+++ felix/trunk/framework/src/main/java/org/apache/felix/framework/BundleImpl.java Tue Feb  3 16:41:58 2009
@@ -38,7 +38,7 @@
 
     private final BundleArchive m_archive;
     private IModule[] m_modules = new IModule[0];
-    private int m_state;
+    private volatile int m_state;
     private BundleActivator m_activator = null;
     private BundleContext m_context = null;
     private final Map m_cachedHeaders = new HashMap();
@@ -581,12 +581,12 @@
         return getFramework().getBundleServicesInUse(this);
     }
 
-    public synchronized int getState()
+    public int getState()
     {
         return m_state;
     }
 
-    synchronized void setState(int i)
+    void setState(int i)
     {
         m_state = i;
     }