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; }