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

Posted to commits@mahout.apache.org by ro...@apache.org on 2010/02/22 19:42:55 UTC

svn commit: r915012 - /lucene/mahout/trunk/math/src/main/java/org/apache/mahout/math/RandomAccessSparseVector.java

Author: robinanil
Date: Mon Feb 22 18:42:55 2010
New Revision: 915012

URL: http://svn.apache.org/viewvc?rev=915012&view=rev
Log:
MAHOUT-300 removing redundant check might speed things a bit

Modified:
    lucene/mahout/trunk/math/src/main/java/org/apache/mahout/math/RandomAccessSparseVector.java

Modified: lucene/mahout/trunk/math/src/main/java/org/apache/mahout/math/RandomAccessSparseVector.java
URL: http://svn.apache.org/viewvc/lucene/mahout/trunk/math/src/main/java/org/apache/mahout/math/RandomAccessSparseVector.java?rev=915012&r1=915011&r2=915012&view=diff
==============================================================================
--- lucene/mahout/trunk/math/src/main/java/org/apache/mahout/math/RandomAccessSparseVector.java (original)
+++ lucene/mahout/trunk/math/src/main/java/org/apache/mahout/math/RandomAccessSparseVector.java Mon Feb 22 18:42:55 2010
@@ -276,7 +276,7 @@
     if(this == x) return dotSelf();
     
     double result = 0;
-    if (x instanceof SequentialAccessSparseVector || x instanceof RandomAccessSparseVector) {
+    if (x instanceof SequentialAccessSparseVector) {
       Iterator<org.apache.mahout.math.Vector.Element> iter = x.iterateNonZero();
       while (iter.hasNext()) {
         org.apache.mahout.math.Vector.Element element = iter.next();