You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by do...@apache.org on 2010/02/23 16:44:46 UTC

svn commit: r915395 - /ofbiz/trunk/framework/base/src/org/ofbiz/base/util/collections/FlexibleMapAccessor.java

Author: doogie
Date: Tue Feb 23 15:44:46 2010
New Revision: 915395

URL: http://svn.apache.org/viewvc?rev=915395&view=rev
Log:
Reworked cache access, it's not important to prevent a multiple
instances from being created.

Modified:
    ofbiz/trunk/framework/base/src/org/ofbiz/base/util/collections/FlexibleMapAccessor.java

Modified: ofbiz/trunk/framework/base/src/org/ofbiz/base/util/collections/FlexibleMapAccessor.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/util/collections/FlexibleMapAccessor.java?rev=915395&r1=915394&r2=915395&view=diff
==============================================================================
--- ofbiz/trunk/framework/base/src/org/ofbiz/base/util/collections/FlexibleMapAccessor.java (original)
+++ ofbiz/trunk/framework/base/src/org/ofbiz/base/util/collections/FlexibleMapAccessor.java Tue Feb 23 15:44:46 2010
@@ -85,11 +85,8 @@
         FlexibleMapAccessor fma = fmaCache.get(original);
         if (fma == null) {
             synchronized (fmaCache) {
+                fmaCache.put(original, new FlexibleMapAccessor(original));
                 fma = fmaCache.get(original);
-                if (fma == null) {
-                    fma = new FlexibleMapAccessor(original);
-                    fmaCache.put(original, fma);
-                }
             }
         }
         return fma;