[cleanup][DenseMatrix] Use method size to obtain vector length in bounds checks.

This allows to use std::vector in methods like DenseMatrix::mv even with DUNE_CHECK_BOUNDS enabled.

added bugfix label

assigned to @simon.praetorius

See #4 for an extensive discussion of the topic. There, most people seem to agree that replacing x.N() by x.size() is a good idea, but @markus.blatt brought up the case of compressed_base_array_unmanaged where this would be problematic. However, this class was meanwhile declared as an implementation detail of dune-istl and should thus not be a problem anymore.

Regarding your patch: At least Dune::Impl::ColumnVectorView is currently missing the size method.

approved this merge request

unapproved this merge request