Resolve #523 9 Correct "copy" constructor from a Matrix. We can just …

…clone the realmatrix.
cdk · Jan 2, 2019 · 6a4cbc1 · 6a4cbc1
1 parent b5e9bd5
commit 6a4cbc1
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/legacy/src/main/java/org/openscience/cdk/math/IMatrix.java b/legacy/src/main/java/org/openscience/cdk/math/IMatrix.java
@@ -61,12 +61,8 @@ public IMatrix(int rows, int columns) {
     public IMatrix(Matrix m) {
         rows = m.rows;
         columns = m.columns;
-        int i, j;
-        for (i = 0; i < rows; i++)
-            for (j = 0; j < columns; j++) {
-                realmatrix[i][j] = m.matrix[i][j];
-                imagmatrix[i][j] = 0d;
-            }
+        realmatrix = m.matrix.clone();
+        imagmatrix = new double[rows][columns];
     }
 
     /**