You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mahout.apache.org by td...@apache.org on 2011/01/29 06:57:25 UTC
svn commit: r1064960 - /mahout/trunk/math/src/main/java/org/apache/mahout/math/AbstractMatrix.java

Author: tdunning
Date: Sat Jan 29 05:57:24 2011
New Revision: 1064960

URL: http://svn.apache.org/viewvc?rev=1064960&view=rev
Log:
MAHOUT-599 - Switched to like() instead of clone() in AbstractMatrix

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

Modified: mahout/trunk/math/src/main/java/org/apache/mahout/math/AbstractMatrix.java
URL: http://svn.apache.org/viewvc/mahout/trunk/math/src/main/java/org/apache/mahout/math/AbstractMatrix.java?rev=1064960&r1=1064959&r2=1064960&view=diff
==============================================================================
--- mahout/trunk/math/src/main/java/org/apache/mahout/math/AbstractMatrix.java (original)
+++ mahout/trunk/math/src/main/java/org/apache/mahout/math/AbstractMatrix.java Sat Jan 29 05:57:24 2011
@@ -387,11 +387,11 @@ public abstract class AbstractMatrix imp
   }
 
   public Matrix divide(double x) {
-    Matrix result = clone();
+    Matrix result = like();
     int[] c = size();
     for (int row = 0; row < c[ROW]; row++) {
       for (int col = 0; col < c[COL]; col++) {
-        result.setQuick(row, col, result.getQuick(row, col) / x);
+        result.setQuick(row, col, getQuick(row, col) / x);
       }
     }
     return result;
@@ -417,10 +417,10 @@ public abstract class AbstractMatrix imp
     if (c[COL] != o[COL]) {
       throw new CardinalityException(c[COL], o[COL]);
     }
-    Matrix result = clone();
+    Matrix result = like();
     for (int row = 0; row < c[ROW]; row++) {
       for (int col = 0; col < c[COL]; col++) {
-        result.setQuick(row, col, result.getQuick(row, col)
+        result.setQuick(row, col, getQuick(row, col)
             - other.getQuick(row, col));
       }
     }
@@ -428,11 +428,11 @@ public abstract class AbstractMatrix imp
   }
 
   public Matrix plus(double x) {
-    Matrix result = clone();
+    Matrix result = like();
     int[] c = size();
     for (int row = 0; row < c[ROW]; row++) {
       for (int col = 0; col < c[COL]; col++) {
-        result.setQuick(row, col, result.getQuick(row, col) + x);
+        result.setQuick(row, col, getQuick(row, col) + x);
       }
     }
     return result;
@@ -447,10 +447,10 @@ public abstract class AbstractMatrix imp
     if (c[COL] != o[COL]) {
       throw new CardinalityException(c[COL], o[COL]);
     }
-    Matrix result = clone();
+    Matrix result = like();
     for (int row = 0; row < c[ROW]; row++) {
       for (int col = 0; col < c[COL]; col++) {
-        result.setQuick(row, col, result.getQuick(row, col)
+        result.setQuick(row, col, getQuick(row, col)
             + other.getQuick(row, col));
       }
     }
@@ -483,11 +483,11 @@ public abstract class AbstractMatrix imp
   }
 
   public Matrix times(double x) {
-    Matrix result = clone();
+    Matrix result = like();
     int[] c = size();
     for (int row = 0; row < c[ROW]; row++) {
       for (int col = 0; col < c[COL]; col++) {
-        result.setQuick(row, col, result.getQuick(row, col) * x);
+        result.setQuick(row, col, getQuick(row, col) * x);
       }
     }
     return result;