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 2013/06/13 06:13:31 UTC

svn commit: r1492524 - /mahout/trunk/math/src/main/java/org/apache/mahout/math/SequentialAccessSparseVector.java

Author: robinanil
Date: Thu Jun 13 04:13:30 2013
New Revision: 1492524

URL: http://svn.apache.org/r1492524
Log:
MAHOUT-1259 toString() method of SequentialAccessSparseVector has closing brace missing for empty vector (by Abhinav M Kulkarni)

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

Modified: mahout/trunk/math/src/main/java/org/apache/mahout/math/SequentialAccessSparseVector.java
URL: http://svn.apache.org/viewvc/mahout/trunk/math/src/main/java/org/apache/mahout/math/SequentialAccessSparseVector.java?rev=1492524&r1=1492523&r2=1492524&view=diff
==============================================================================
--- mahout/trunk/math/src/main/java/org/apache/mahout/math/SequentialAccessSparseVector.java (original)
+++ mahout/trunk/math/src/main/java/org/apache/mahout/math/SequentialAccessSparseVector.java Thu Jun 13 04:13:30 2013
@@ -136,9 +136,7 @@ public class SequentialAccessSparseVecto
       result.append(e.get());
       result.append(',');
     }
-    if (result.length() > 1) {
-      result.setCharAt(result.length() - 1, '}');
-    }
+    result.append('}');
     return result.toString();
   }