From 036f4ed9e95f1aa90c3e5ad57aba6bb5308444e1 Mon Sep 17 00:00:00 2001 From: Wolfgang Bangerth Date: Mon, 17 Jul 2017 17:11:20 -0600 Subject: [PATCH] Use a clearer initialization syntax. Specifically, we used the syntax Type object = initializer; in contexts where the initializer is of a different type and the compiler automatically converted this into Type object(initializer); where the constructor called was just a regular constructor that did something with the initializer, but specifically did not copy the object. This is confusing. Use the latter syntax instead. --- include/deal.II/lac/relaxation_block.templates.h | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/include/deal.II/lac/relaxation_block.templates.h b/include/deal.II/lac/relaxation_block.templates.h index c0b7d03ab6..63deb43a73 100644 --- a/include/deal.II/lac/relaxation_block.templates.h +++ b/include/deal.II/lac/relaxation_block.templates.h @@ -279,7 +279,7 @@ void RelaxationBlockJacobi::step const VectorType &src) const { GrowingVectorMemory mem; - typename VectorMemory::Pointer aux = mem; + typename VectorMemory::Pointer aux(mem); aux->reinit(dst, false); *aux = dst; this->do_step(dst, *aux, src, false); @@ -292,7 +292,7 @@ void RelaxationBlockJacobi::Tstep const VectorType &src) const { GrowingVectorMemory mem; - typename VectorMemory::Pointer aux = mem; + typename VectorMemory::Pointer aux(mem); aux->reinit(dst, false); *aux = dst; this->do_step(dst, *aux, src, true); @@ -305,7 +305,7 @@ void RelaxationBlockJacobi::vmult const VectorType &src) const { GrowingVectorMemory mem; - typename VectorMemory::Pointer aux = mem; + typename VectorMemory::Pointer aux(mem); dst = 0; aux->reinit(dst); this->do_step(dst, *aux, src, false); @@ -318,7 +318,7 @@ void RelaxationBlockJacobi::Tvmult const VectorType &src) const { GrowingVectorMemory mem; - typename VectorMemory::Pointer aux = mem; + typename VectorMemory::Pointer aux(mem); dst = 0; aux->reinit(dst); this->do_step(dst, *aux, src, true); -- 2.39.5