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/04/01 06:40:39 UTC

svn commit: r929829 - /ofbiz/trunk/framework/base/src/org/ofbiz/base/util/cache/UtilCache.java

Author: doogie
Date: Thu Apr  1 04:40:39 2010
New Revision: 929829

URL: http://svn.apache.org/viewvc?rev=929829&view=rev
Log:
Refactor createCacheLine slightly.

Modified:
    ofbiz/trunk/framework/base/src/org/ofbiz/base/util/cache/UtilCache.java

Modified: ofbiz/trunk/framework/base/src/org/ofbiz/base/util/cache/UtilCache.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/util/cache/UtilCache.java?rev=929829&r1=929828&r2=929829&view=diff
==============================================================================
--- ofbiz/trunk/framework/base/src/org/ofbiz/base/util/cache/UtilCache.java (original)
+++ ofbiz/trunk/framework/base/src/org/ofbiz/base/util/cache/UtilCache.java Thu Apr  1 04:40:39 2010
@@ -232,10 +232,11 @@ public class UtilCache<K, V> implements 
     }
 
     private CacheLine<V> createCacheLine(V value, long expireTime) {
-        if (expireTime > 0) {
-            return useSoftReference ? new SoftRefCacheLine<V>(value, System.currentTimeMillis(), expireTime) : new HardRefCacheLine<V>(value, System.currentTimeMillis(), expireTime);
+        long loadTime = expireTime > 0 ? System.currentTimeMillis() : 0;
+        if (useSoftReference) {
+            return new SoftRefCacheLine<V>(value, loadTime, expireTime);
         } else {
-            return useSoftReference ? new SoftRefCacheLine<V>(value, 0, expireTime) : new HardRefCacheLine<V>(value, 0, expireTime);
+            return new HardRefCacheLine<V>(value, loadTime, expireTime);
         }
     }