[Dune-devel] [Dune-Commit] [Commit] dune-istl - dd0f754: [BCRSMatrix] Improve error reporting in BCRSMatrix

[Markus Blatt]

Hi,

ImplicitModeOverflowExhausted makes sense. What I am wondering is
whether some common exception for running out of memory (that should
be used in the other build modes if we run out of memory) would
suffice and make even more sense.

What I fail to see any use in having yet another exception
BCRSMatrixMatrix just for problems in BCRSMatrix. This seems to be
rather uncommon and what mean that we might end up with one exception
per class.

Markus

On Tue, Nov 19, 2013 at 02:09:48PM +0100, Steffen Müthing wrote:
> New commit, appeared at Tue Nov 19 14:09:48 2013 +0100
> as part of the following ref changes:
> 
>     branch refs/heads/feature/new-bcrsmatrix-buildmode    created as              b9bbcfa
> 
> Browsable version: http://cgit.dune-project.org/repositories/dune-istl/commit/?id=dd0f754c08374bdda40d46d162def8a94bb8604f
> 
> ======================================================================
> 
> commit dd0f754c08374bdda40d46d162def8a94bb8604f
> Author: Steffen Müthing <muething at dune-project.org>
> Date:   Fri Nov 15 19:10:59 2013 +0100
> 
>     [BCRSMatrix] Improve error reporting in BCRSMatrix
>     
>     - Introduce new exception BCRSMatrixError and use it throughout
>       BCRSMatrix
>     - Introduce new exception ImplicitModeOverflowExhausted, which is thrown
>       by compress() for the implicit build mode. Also improve the error
>       message for that exception to really help the user understand the
>       problem signalled by this exception and how to remedy the situation.
>