From 93d8d0e1b528aa0f89075b38132da14b2d814cd6 Mon Sep 17 00:00:00 2001 From: Benoit Jacob Date: Wed, 9 Jun 2010 20:50:18 -0400 Subject: [PATCH] add unit test for bug #132 --- test/CMakeLists.txt | 1 + test/bug_132.cpp | 41 +++++++++++++++++++++++++++++++++++++++++ 2 files changed, 42 insertions(+) create mode 100644 test/bug_132.cpp diff --git a/test/CMakeLists.txt b/test/CMakeLists.txt index 93588ee8e..bac3f4877 100644 --- a/test/CMakeLists.txt +++ b/test/CMakeLists.txt @@ -236,6 +236,7 @@ if(NOT EIGEN_DEFAULT_TO_ROW_MAJOR) endif() ei_add_test(swap) ei_add_test(visitor) +ei_add_test(bug_132) ei_add_test(prec_inverse_4x4 ${EI_OFLAG}) diff --git a/test/bug_132.cpp b/test/bug_132.cpp new file mode 100644 index 000000000..aa9bf2c28 --- /dev/null +++ b/test/bug_132.cpp @@ -0,0 +1,41 @@ +// This file is part of Eigen, a lightweight C++ template library +// for linear algebra. Eigen itself is part of the KDE project. +// +// Copyright (C) 2010 Benoit Jacob +// +// Eigen is free software; you can redistribute it and/or +// modify it under the terms of the GNU Lesser General Public +// License as published by the Free Software Foundation; either +// version 3 of the License, or (at your option) any later version. +// +// Alternatively, you can redistribute it and/or +// modify it under the terms of the GNU General Public License as +// published by the Free Software Foundation; either version 2 of +// the License, or (at your option) any later version. +// +// Eigen is distributed in the hope that it will be useful, but WITHOUT ANY +// WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS +// FOR A PARTICULAR PURPOSE. See the GNU Lesser General Public License or the +// GNU General Public License for more details. +// +// You should have received a copy of the GNU Lesser General Public +// License and a copy of the GNU General Public License along with +// Eigen. If not, see . + +#include "main.h" + +void test_bug_132() { + enum { size = 100 }; + MatrixXd A(size, size); + VectorXd b(size), c(size); + { + VectorXd y = A.transpose() * (b-c); // bug 132: infinite recursion in coeffRef + VectorXd z = (b-c).transpose() * A; // bug 132: infinite recursion in coeffRef + } + + // the following ones weren't failing, but let's include them for completeness: + { + VectorXd y = A * (b-c); + VectorXd z = (b-c).transpose() * A.transpose(); + } +}