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

Posted to commits@shiro.apache.org by lh...@apache.org on 2010/01/14 21:27:16 UTC

svn commit: r899400 - /incubator/shiro/trunk/core/src/main/java/org/apache/shiro/cache/DefaultCacheManager.java

Author: lhazlewood
Date: Thu Jan 14 20:27:12 2010
New Revision: 899400

URL: http://svn.apache.org/viewvc?rev=899400&view=rev
Log:
Changed the destroy method to be a more concurrent-friendly implementation

Modified:
    incubator/shiro/trunk/core/src/main/java/org/apache/shiro/cache/DefaultCacheManager.java

Modified: incubator/shiro/trunk/core/src/main/java/org/apache/shiro/cache/DefaultCacheManager.java
URL: http://svn.apache.org/viewvc/incubator/shiro/trunk/core/src/main/java/org/apache/shiro/cache/DefaultCacheManager.java?rev=899400&r1=899399&r2=899400&view=diff
==============================================================================
--- incubator/shiro/trunk/core/src/main/java/org/apache/shiro/cache/DefaultCacheManager.java (original)
+++ incubator/shiro/trunk/core/src/main/java/org/apache/shiro/cache/DefaultCacheManager.java Thu Jan 14 20:27:12 2010
@@ -64,7 +64,7 @@
     }
 
     public void destroy() throws Exception {
-        synchronized (caches) {
+        while( !caches.isEmpty() ) {
             for (Cache cache : caches.values()) {
                 LifecycleUtils.destroy(cache);
             }