From 735faa2a88a5965bf31a6d8c100005f26b96e86a Mon Sep 17 00:00:00 2001
From: William F Godoy <williamfgc@yahoo.com>
Date: Mon, 5 Jun 2017 17:13:47 -0400
Subject: [PATCH] Removed MPI_COMM_SELF from headers, fixed bug in VariableBase

---
 source/adios2/core/IO.h                             | 2 +-
 source/adios2/core/VariableBase.cpp                 | 4 ++--
 source/adios2/toolkit/interop/adios1/ADIOS1Common.h | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/source/adios2/core/IO.h b/source/adios2/core/IO.h
index 215dbbfd9..87df704d8 100644
--- a/source/adios2/core/IO.h
+++ b/source/adios2/core/IO.h
@@ -42,7 +42,7 @@ public:
 
     /** from ADIOS class passed to Engine created with Open
      *  if no new communicator is passed */
-    MPI_Comm m_MPIComm = MPI_COMM_SELF;
+    MPI_Comm m_MPIComm;
 
     /** true: extra exceptions checks */
     const bool m_DebugMode = false;
diff --git a/source/adios2/core/VariableBase.cpp b/source/adios2/core/VariableBase.cpp
index 40632a61e..db50a21db 100644
--- a/source/adios2/core/VariableBase.cpp
+++ b/source/adios2/core/VariableBase.cpp
@@ -112,8 +112,8 @@ void VariableBase::SetMemorySelection(const SelectionBoundingBox &selection)
         }
     }
 
-    ConvertUint64VectorToSizetVector(selection.m_Count, m_MemoryCount);
-    ConvertUint64VectorToSizetVector(selection.m_Start, m_MemoryStart);
+    m_MemoryCount = selection.m_Count;
+    m_MemoryStart = selection.m_Start;
 }
 
 void VariableBase::SetStepSelection(const unsigned int startStep,
diff --git a/source/adios2/toolkit/interop/adios1/ADIOS1Common.h b/source/adios2/toolkit/interop/adios1/ADIOS1Common.h
index 05327d92b..c5e2297f4 100644
--- a/source/adios2/toolkit/interop/adios1/ADIOS1Common.h
+++ b/source/adios2/toolkit/interop/adios1/ADIOS1Common.h
@@ -73,7 +73,7 @@ public:
     void Close();
 
 private:
-    MPI_Comm m_MPIComm = MPI_COMM_SELF;
+    MPI_Comm m_MPIComm;
     const bool m_DebugMode = false;
 
     void Init();
-- 
GitLab