[Dune] [#1127] MatrixDimension<MType>::coldim fails for MType=BCRSMatrix<Diagonal|ScaledIDMatrix> (Attachment added)

[Dune]

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.

A new Flyspray task has been opened.  Details are below. 

User who did this - Uli Sack (usack) 

Attached to Project - Dune
Summary - MatrixDimension<MType>::coldim fails for MType=BCRSMatrix<Diagonal|ScaledIDMatrix> 
Task Type - Bug Report
Category - ISTL
Status - New
Assigned To - 
Operating System - Unspecified / All
Severity - Low
Priority - Normal
Reported Version - SVN (pre2.3)
Due in Version - Undecided
Due Date - Undecided
Details - MatrixDimension<MType>::coldim fails for MType=BCRSMatrix<[DiagonalMatrix|ScaledIDMatrix]> and MType=DiagonalMatrix|ScaledIDMatrix]. The latter is due to a missing template specialization for these block types. The former is due to the use of operator[] on a shared_ptr on line 256. Using get()[] and implementing the specializations gets rid of both problems. Find a patch attached.

However I don't quite get the intended functionality. In the partial specialization template for MType=BCRSMatrix<B,A> coldim(MType A, size_type c) returns 0 if A has no entry in column c whereas the specialization for BCRSMatrix<FieldMatrix<K,n,m> > invariably returns m no matter whether any entry has the column index c or not. Is that intended or simply inconsistent?

Cheers,Uli

One or more files have been attached.

More information can be found at the following URL:
http://www.dune-project.org/flyspray/index.php?do=details&task_id=1127

You are receiving this message because you have requested it from the Flyspray bugtracking system.  If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.