From d86f5339b2033af792fa60176f8060938e9599ec Mon Sep 17 00:00:00 2001 From: Jitse Niesen Date: Fri, 26 Feb 2010 09:47:17 +0000 Subject: ComplexSchur: fix bug introduced in my previous commit. The value of c is actually used a few lines later. --- Eigen/src/Eigenvalues/ComplexSchur.h | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'Eigen') diff --git a/Eigen/src/Eigenvalues/ComplexSchur.h b/Eigen/src/Eigenvalues/ComplexSchur.h index 0fad415a2..531ebf709 100644 --- a/Eigen/src/Eigenvalues/ComplexSchur.h +++ b/Eigen/src/Eigenvalues/ComplexSchur.h @@ -218,10 +218,12 @@ void ComplexSchur::compute(const MatrixType& matrix, bool skipU) sf = t.cwiseAbs().sum(); t /= sf; // the normalization by sf is to avoid under/overflow - b = t.coeff(0,0) + t.coeff(1,1); + b = t.coeff(0,1) * t.coeff(1,0); c = t.coeff(0,0) - t.coeff(1,1); - disc = ei_sqrt(c*c + RealScalar(4)*t.coeff(0,1)*t.coeff(1,0)); + disc = ei_sqrt(c*c + RealScalar(4)*b); + c = t.coeff(0,0) * t.coeff(1,1) - b; + b = t.coeff(0,0) + t.coeff(1,1); r1 = (b+disc)/RealScalar(2); r2 = (b-disc)/RealScalar(2); -- cgit v1.2.3