Simplify implementation for vectors with scalar entries
Using the new Impl::asVector()
from dune-common we can get rid of
- several
Hybrid::isElse()
constructions used to implement special cases forisNumber
, - undefined behaviour invoked by
toVector()
due to the latter violating strict aliasing rules.
This also removed the dangerous toVector()
methods. This is only a partial solution to #63 (closed). We also need to do the same for toMatrix()
which requires an analogue Impl::asMatrix()
function and a ScalarMatrixView
wrapper class.
Edited by Oliver Sander
Merge request reports
Activity
assigned to @oliver.sander
added 6 commits
-
5c11a9b6...37311d14 - 3 commits from branch
master
- 0e962e10 - Simplify scalar entries in block vector classes
- 36190383 - Add missing include
- d883ca6e - Get rid of toVector()
Toggle commit list-
5c11a9b6...37311d14 - 3 commits from branch
enabled an automatic merge when the pipeline for d883ca6e succeeds
mentioned in commit 39995de5
Please register or sign in to reply