Skip to content
Snippets Groups Projects
Commit 8cd7d792 authored by Steffen Müthing's avatar Steffen Müthing
Browse files

[!281] Don't spam people with warnings they can't do anything about

Merge branch 'bugfix/silence-useless-warnings' into 'master'

See merge request [!281]

  [!281]: Nonecore/dune-istl/merge_requests/281
parents 1a90fd2c 30c3f1f1
Branches
Tags
1 merge request!281Don't spam people with warnings they can't do anything about
Pipeline #17151 passed
......@@ -276,7 +276,7 @@ namespace Dune
}
//! \brief deep copy of a given hierarchy
#warning do we catually want to support this? This might be very expensive?!
//TODO: do we actually want to support this? This might be very expensive?!
template<class T, class A>
Hierarchy<T,A>::Hierarchy(const Hierarchy& other)
: allocator_(other.allocator_),
......@@ -297,7 +297,7 @@ namespace Dune
// create shared_ptr from weak_ptr, we just checked that this is safe
std::shared_ptr<Element> otherCurrent_ = std::shared_ptr<Element>(otherWeak_);
// clone current level
#warning should we use the allocator?
//TODO: should we use the allocator?
current_->element_ =
std::make_shared<MemberType>(*(otherCurrent_->element_));
current_->finer_=finer_;
......@@ -354,7 +354,7 @@ namespace Dune
template<class T, class A>
void Hierarchy<T,A>::addFiner(Arguments& args)
{
#warning wouldn't it be better to do this in the constructor?'
//TODO: wouldn't it be better to do this in the constructor?'
if(!finest_) {
// we have no levels at all...
assert(!coarsest_);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment