Author: jochen
Date: Tue Mar  1 22:35:01 2011
New Revision: 1076039

URL: http://svn.apache.org/viewvc?rev=1076039&view=rev
Log:
PR: COLLECTIONS-334
Synchronizing access to locks in StaticBucketMap.size()

Modified:
    
commons/proper/collections/branches/COLLECTIONS_3_2_BRANCH/src/java/org/apache/commons/collections/map/StaticBucketMap.java
    
commons/proper/collections/trunk/src/java/org/apache/commons/collections/map/StaticBucketMap.java

Modified: 
commons/proper/collections/branches/COLLECTIONS_3_2_BRANCH/src/java/org/apache/commons/collections/map/StaticBucketMap.java
URL: 
http://svn.apache.org/viewvc/commons/proper/collections/branches/COLLECTIONS_3_2_BRANCH/src/java/org/apache/commons/collections/map/StaticBucketMap.java?rev=1076039&r1=1076038&r2=1076039&view=diff
==============================================================================
--- 
commons/proper/collections/branches/COLLECTIONS_3_2_BRANCH/src/java/org/apache/commons/collections/map/StaticBucketMap.java
 (original)
+++ 
commons/proper/collections/branches/COLLECTIONS_3_2_BRANCH/src/java/org/apache/commons/collections/map/StaticBucketMap.java
 Tue Mar  1 22:35:01 2011
@@ -182,7 +182,9 @@ public final class StaticBucketMap imple
         int cnt = 0;
 
         for (int i = 0; i < buckets.length; i++) {
-            cnt += locks[i].size;
+            synchronized(locks[i]) {
+                cnt += locks[i].size;
+            }
         }
         return cnt;
     }

Modified: 
commons/proper/collections/trunk/src/java/org/apache/commons/collections/map/StaticBucketMap.java
URL: 
http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/java/org/apache/commons/collections/map/StaticBucketMap.java?rev=1076039&r1=1076038&r2=1076039&view=diff
==============================================================================
--- 
commons/proper/collections/trunk/src/java/org/apache/commons/collections/map/StaticBucketMap.java
 (original)
+++ 
commons/proper/collections/trunk/src/java/org/apache/commons/collections/map/StaticBucketMap.java
 Tue Mar  1 22:35:01 2011
@@ -183,7 +183,9 @@ public final class StaticBucketMap<K, V>
         int cnt = 0;
 
         for (int i = 0; i < buckets.length; i++) {
-            cnt += locks[i].size;
+            synchronized(locks[i]) {
+                cnt += locks[i].size;
+            }
         }
         return cnt;
     }


Reply via email to