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

svn commit: r748184 - /lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/common/FastMap.java

Author: srowen
Date: Thu Feb 26 15:44:20 2009
New Revision: 748184

URL: http://svn.apache.org/viewvc?rev=748184&view=rev
Log:
Fixed FastMap bug due to new BitSet -- need to set its capacity larger

Modified:
    lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/common/FastMap.java

Modified: lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/common/FastMap.java
URL: http://svn.apache.org/viewvc/lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/common/FastMap.java?rev=748184&r1=748183&r2=748184&view=diff
==============================================================================
--- lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/common/FastMap.java (original)
+++ lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/common/FastMap.java Thu Feb 26 15:44:20 2009
@@ -52,7 +52,7 @@
   private int numEntries;
   private int numSlotsUsed;
   private int maxSize;
-  private final BitSet recentlyAccessed;
+  private BitSet recentlyAccessed;
   private final boolean countingAccesses;
 
   /**
@@ -90,7 +90,7 @@
     values = (V[]) new Object[hashSize];
     this.maxSize = maxSize;
     this.countingAccesses = maxSize != Integer.MAX_VALUE;
-    this.recentlyAccessed = countingAccesses ? new BitSet(maxSize) : null;
+    this.recentlyAccessed = countingAccesses ? new BitSet(hashSize) : null;
   }
 
   /**
@@ -301,7 +301,7 @@
     numEntries = 0;
     numSlotsUsed = 0;
     if (countingAccesses) {
-      recentlyAccessed.clear();
+      recentlyAccessed = new BitSet(newHashSize);
     }
     keys = (K[]) new Object[newHashSize];
     values = (V[]) new Object[newHashSize];