From 204a9a2300dfbc135783a02ad41bb72368740a26 Mon Sep 17 00:00:00 2001
From: Oliver Sander <sander@igpm.rwth-aachen.de>
Date: Thu, 16 Oct 2014 13:58:26 +0000
Subject: [PATCH] Remove the code sharing local sizes from the constructor
That code was buggy, and we only didn't see that because the wrong sizes
were always overwritten by the correct ones before being used.
(That's so as long as you call a reducing method before 'scatter').
Removing the code leaves the situation as bad as before, but it
allows us to get rid of the nOwnedLocalEntity() method from the global
mapper.
[[Imported from SVN: r9926]]
---
dune/gfe/parallel/vectorcommunicator.hh | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/dune/gfe/parallel/vectorcommunicator.hh b/dune/gfe/parallel/vectorcommunicator.hh
index c88de90d..353951dc 100644
--- a/dune/gfe/parallel/vectorcommunicator.hh
+++ b/dune/gfe/parallel/vectorcommunicator.hh
@@ -43,10 +43,7 @@ public:
const Communicator& communicator,
const int& root)
: guIndex(gi), communicator_(communicator), root_rank(root)
- {
- // Get number of vector entries on each process
- localVectorEntriesSizes = MPIFunctions::shareSizes(communicator, guIndex.nOwnedLocalEntity());
- }
+ {}
VectorType reduceAdd(const VectorType& localVector)
{
--
GitLab