You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@openwebbeans.apache.org by st...@apache.org on 2011/03/22 19:39:47 UTC

svn commit: r1084291 - /openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/context/creational/CreationalContextImpl.java

Author: struberg
Date: Tue Mar 22 18:39:47 2011
New Revision: 1084291

URL: http://svn.apache.org/viewvc?rev=1084291&view=rev
Log:
removed TODOs because ReentrantLock was slower than synchronized

Modified:
    openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/context/creational/CreationalContextImpl.java

Modified: openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/context/creational/CreationalContextImpl.java
URL: http://svn.apache.org/viewvc/openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/context/creational/CreationalContextImpl.java?rev=1084291&r1=1084290&r2=1084291&view=diff
==============================================================================
--- openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/context/creational/CreationalContextImpl.java (original)
+++ openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/context/creational/CreationalContextImpl.java Tue Mar 22 18:39:47 2011
@@ -175,7 +175,6 @@ public class CreationalContextImpl<T> im
 
             dependentCreational.setInstance(instance);
 
-            //X TODO maybe better with ReentrantReadWriteLock?
             synchronized(this)
             {
                 if (dependentObjects == null)
@@ -209,7 +208,6 @@ public class CreationalContextImpl<T> im
             return null;
         }
 
-        //X TODO maybe better with ReentrantReadWriteLock?
         synchronized(this)
         {
             List<DependentCreationalContext<?>> values = this.dependentObjects.get(ownerInstance);