From 4f660b1ec53762c51e29a8ce1e6c54b9ab6a20d4 Mon Sep 17 00:00:00 2001 From: Lukas Riedel Date: Wed, 6 Feb 2019 15:26:45 +0100 Subject: [PATCH] Remove assertions from flux reconstruction functions The constructors properly initialize the asserted objects. There is no point in the code where they can be replaced by empty/null pointers. --- dune/dorie/model/richards/richards.hh | 8 -------- dune/dorie/model/transport/transport.hh | 6 ------ 2 files changed, 14 deletions(-) diff --git a/dune/dorie/model/richards/richards.hh b/dune/dorie/model/richards/richards.hh index e01b1dc2..ec5539f9 100644 --- a/dune/dorie/model/richards/richards.hh +++ b/dune/dorie/model/richards/richards.hh @@ -559,10 +559,6 @@ public: gf_ptr = std::make_unique( this->_log,gv,inifile.sub("fluxReconstruction")); - assert(fboundary); - assert(fparam); - assert(slop); - slop->setTime(state.time); // update it with the state @@ -579,10 +575,6 @@ public: cache = std::make_unique( this->_log,gv,inifile.sub("fluxReconstruction")); - assert(fboundary); - assert(fparam); - assert(slop); - // update it with current state cache->update(*u,*gfs,*slop); diff --git a/dune/dorie/model/transport/transport.hh b/dune/dorie/model/transport/transport.hh index 9efcba44..de7fdd91 100644 --- a/dune/dorie/model/transport/transport.hh +++ b/dune/dorie/model/transport/transport.hh @@ -488,9 +488,6 @@ public: gf_ptr = std::make_unique( this->_log,gv,inifile.sub("fluxReconstruction")); - - assert(sboundary); - assert(slop); slop->setTime(state.time); @@ -505,9 +502,6 @@ public: cache = std::make_unique( this->_log,gv,inifile.sub("fluxReconstruction")); - - assert(sboundary); - assert(slop); slop->setTime(state.time); -- GitLab