From 17b847f8915e7f4cb51b043efe811f34dd15243c Mon Sep 17 00:00:00 2001 From: Christoph Ruegg Date: Fri, 18 Jun 2010 16:17:35 +0200 Subject: [PATCH] Tests: User-defined Matrix/Vector now copy provided array in ctor, fixing failing LA Permutation unit tests --- src/Numerics/LinearAlgebra/Double/Matrix.cs | 4 ++-- .../LinearAlgebraTests/Double/UserDefinedMatrixTests.cs | 2 +- .../LinearAlgebraTests/Double/UserDefinedVectorTests.cs | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/Numerics/LinearAlgebra/Double/Matrix.cs b/src/Numerics/LinearAlgebra/Double/Matrix.cs index 15512d4f..e7eda7da 100644 --- a/src/Numerics/LinearAlgebra/Double/Matrix.cs +++ b/src/Numerics/LinearAlgebra/Double/Matrix.cs @@ -1498,7 +1498,7 @@ namespace MathNet.Numerics.LinearAlgebra.Double // Get a sequence of inversions from the permutation. var inv = p.ToInversions(); - for (var i = 0; i < p.Dimension; i++) + for (var i = 0; i < inv.Length; i++) { if (inv[i] != i) { @@ -1527,7 +1527,7 @@ namespace MathNet.Numerics.LinearAlgebra.Double // Get a sequence of inversions from the permutation. var inv = p.ToInversions(); - for (var i = 0; i < p.Dimension; i++) + for (var i = 0; i < inv.Length; i++) { if (inv[i] != i) { diff --git a/src/UnitTests/LinearAlgebraTests/Double/UserDefinedMatrixTests.cs b/src/UnitTests/LinearAlgebraTests/Double/UserDefinedMatrixTests.cs index e016c8bd..b7ecdae3 100644 --- a/src/UnitTests/LinearAlgebraTests/Double/UserDefinedMatrixTests.cs +++ b/src/UnitTests/LinearAlgebraTests/Double/UserDefinedMatrixTests.cs @@ -43,7 +43,7 @@ namespace MathNet.Numerics.UnitTests.LinearAlgebraTests.Double public UserDefinedMatrix(double[,] data) : base(data.GetLength(0), data.GetLength(1)) { - _data = data; + _data = (double[,])data.Clone(); } public override double At(int row, int column) diff --git a/src/UnitTests/LinearAlgebraTests/Double/UserDefinedVectorTests.cs b/src/UnitTests/LinearAlgebraTests/Double/UserDefinedVectorTests.cs index ffb1eed1..5ab398db 100644 --- a/src/UnitTests/LinearAlgebraTests/Double/UserDefinedVectorTests.cs +++ b/src/UnitTests/LinearAlgebraTests/Double/UserDefinedVectorTests.cs @@ -46,7 +46,7 @@ namespace MathNet.Numerics.UnitTests.LinearAlgebraTests.Double public UserDefinedVector(double[] data) : base(data.Length) { - _data = data; + _data = (double[])data.Clone(); } public override double this[int index]