From fe91b1ba70c2f3042f14cbd75baa2e8dd70f5d65 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 abdb3fe0..8e4e223f 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 36ac045c..06548996 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 89cb295b..505a6671 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: