From 8915d5bd2217d4ebbf8fe54ebd2dfbf3f0c420c4 Mon Sep 17 00:00:00 2001 From: Gael Guennebaud Date: Tue, 1 Feb 2011 17:21:20 +0100 Subject: fix 168 : now TriangularView::solve returns by value making TriangularView::solveInPlace less important. Also fix the very outdated documentation of this function. --- test/product_trsolve.cpp | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'test/product_trsolve.cpp') diff --git a/test/product_trsolve.cpp b/test/product_trsolve.cpp index f9ad049a4..c207cc500 100644 --- a/test/product_trsolve.cpp +++ b/test/product_trsolve.cpp @@ -28,12 +28,18 @@ (XB).setRandom(); ref = (XB); \ (TRI).solveInPlace(XB); \ VERIFY_IS_APPROX((TRI).toDenseMatrix() * (XB), ref); \ + (XB).setRandom(); ref = (XB); \ + (XB) = (TRI).solve(XB); \ + VERIFY_IS_APPROX((TRI).toDenseMatrix() * (XB), ref); \ } #define VERIFY_TRSM_ONTHERIGHT(TRI,XB) { \ (XB).setRandom(); ref = (XB); \ (TRI).transpose().template solveInPlace(XB.transpose()); \ VERIFY_IS_APPROX((XB).transpose() * (TRI).transpose().toDenseMatrix(), ref.transpose()); \ + (XB).setRandom(); ref = (XB); \ + (XB).transpose() = (TRI).transpose().template solve(XB.transpose()); \ + VERIFY_IS_APPROX((XB).transpose() * (TRI).transpose().toDenseMatrix(), ref.transpose()); \ } template void trsolve(int size=Size,int cols=Cols) -- cgit v1.2.3