From 9ed9f8b2885e080254a5fbab1aead3fabad8eede Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Atgeirr=20Fl=C3=B8=20Rasmussen?= <atgeirr@sintef.no>
Date: Tue, 21 Feb 2012 21:54:46 +0100
Subject: [PATCH] Fixed class comments, LinearSolverUmfpack::solve() properly
 returns a report.

---
 opm/core/linalg/LinearSolverIstl.hpp    | 2 +-
 opm/core/linalg/LinearSolverUmfpack.cpp | 3 +++
 opm/core/linalg/LinearSolverUmfpack.hpp | 2 +-
 3 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/opm/core/linalg/LinearSolverIstl.hpp b/opm/core/linalg/LinearSolverIstl.hpp
index abdb3fe08..8e4e223ff 100644
--- a/opm/core/linalg/LinearSolverIstl.hpp
+++ b/opm/core/linalg/LinearSolverIstl.hpp
@@ -30,7 +30,7 @@ namespace Opm
 {
 
 
-    /// Abstract interface for linear solvers.
+    /// Concrete class encapsulating some dune-istl linear solvers.
     class LinearSolverIstl : public LinearSolverInterface
     {
     public:
diff --git a/opm/core/linalg/LinearSolverUmfpack.cpp b/opm/core/linalg/LinearSolverUmfpack.cpp
index 36ac045ca..065489966 100644
--- a/opm/core/linalg/LinearSolverUmfpack.cpp
+++ b/opm/core/linalg/LinearSolverUmfpack.cpp
@@ -55,6 +55,9 @@ namespace Opm
             const_cast<double*>(sa)
         };
         call_UMFPACK(&A, rhs, solution);
+        LinearSolverReport rep;
+        rep.converged = true;
+        return rep;
     }
 
 } // namespace Opm
diff --git a/opm/core/linalg/LinearSolverUmfpack.hpp b/opm/core/linalg/LinearSolverUmfpack.hpp
index 89cb295b9..505a66718 100644
--- a/opm/core/linalg/LinearSolverUmfpack.hpp
+++ b/opm/core/linalg/LinearSolverUmfpack.hpp
@@ -49,7 +49,7 @@ namespace Opm
 {
 
 
-    /// Abstract interface for linear solvers.
+    /// Concrete class encapsulating the UMFPACK direct linear solver.
     class LinearSolverUmfpack : public LinearSolverInterface
     {
     public: