Resolve #523 9 Correct "copy" constructor from a Matrix. We can just clone the realmatrix.

johnmay · johnmay · commit 6a4cbc1ea518 · 2019-01-02T22:24:07.000Z
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];
     }
 
     /**
