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