From 95ee6566d4c105dfd6dc95ab5a0036bd43cba76a Mon Sep 17 00:00:00 2001
From: Arne Morten Kvarving <arne.morten.kvarving@sintef.no>
Date: Tue, 3 Dec 2019 09:20:28 +0100
Subject: [PATCH] added: a macro to handle types as pod

---
 opm/simulators/utils/ParallelRestart.cpp | 78 ++++++------------------
 1 file changed, 19 insertions(+), 59 deletions(-)

diff --git a/opm/simulators/utils/ParallelRestart.cpp b/opm/simulators/utils/ParallelRestart.cpp
index 81369fa21..56b2ed2c4 100644
--- a/opm/simulators/utils/ParallelRestart.cpp
+++ b/opm/simulators/utils/ParallelRestart.cpp
@@ -31,6 +31,22 @@
 #include <opm/parser/eclipse/EclipseState/Tables/TableSchema.hpp>
 #include <dune/common/parallel/mpitraits.hh>
 
+#define HANDLE_AS_POD(T) \
+  std::size_t packSize(const T& data, Dune::MPIHelper::MPICommunicator comm) \
+  { \
+      return packSize(data, comm, std::integral_constant<bool,true>()); \
+  } \
+  void pack(const T& data, std::vector<char>& buffer, int& position, \
+            Dune::MPIHelper::MPICommunicator comm) \
+  { \
+      pack(data, buffer, position, comm, std::integral_constant<bool,true>()); \
+  } \
+  void unpack(T& data, std::vector<char>& buffer, int& position, \
+              Dune::MPIHelper::MPICommunicator comm) \
+  { \
+      unpack(data, buffer, position, comm, std::integral_constant<bool,true>()); \
+  }
+
 namespace Opm
 {
 namespace Mpi
@@ -160,23 +176,9 @@ std::size_t packSize(const std::unordered_map<T1,T2,H,P,A>& data, Dune::MPIHelpe
     return totalSize;
 }
 
-std::size_t packSize(const data::Rates& data, Dune::MPIHelper::MPICommunicator comm)
-{
-    // plain struct no pointers -> treat as pod
-    return packSize(data, comm, std::integral_constant<bool,true>());
-}
-
-std::size_t packSize(const data::Connection& data, Dune::MPIHelper::MPICommunicator comm)
-{
-    // plain struct no pointers -> treat as pod
-    return packSize(data, comm, std::integral_constant<bool,true>());
-}
-
-std::size_t packSize(const data::Segment& data, Dune::MPIHelper::MPICommunicator comm)
-{
-    // plain struct no pointers -> treat as pod
-    return packSize(data, comm, std::integral_constant<bool,true>());
-}
+HANDLE_AS_POD(data::Connection)
+HANDLE_AS_POD(data::Rates)
+HANDLE_AS_POD(data::Segment)
 
 std::size_t packSize(const data::Well& data, Dune::MPIHelper::MPICommunicator comm)
 {
@@ -409,27 +411,6 @@ void pack(const std::unordered_map<T1,T2,H,P,A>& data, std::vector<char>& buffer
     }
 }
 
-void pack(const data::Rates& data, std::vector<char>& buffer, int& position,
-          Dune::MPIHelper::MPICommunicator comm)
-{
-    // plain struct no pointers -> treat as pod
-    pack(data, buffer, position, comm, std::integral_constant<bool,true>());
-}
-
-void pack(const data::Connection& data, std::vector<char>& buffer, int& position,
-          Dune::MPIHelper::MPICommunicator comm)
-{
-    // plain struct no pointers -> treat as pod
-    pack(data, buffer, position, comm, std::integral_constant<bool,true>());
-}
-
-void pack(const data::Segment& data,std::vector<char>& buffer, int& position,
-          Dune::MPIHelper::MPICommunicator comm)
-{
-    // plain struct no pointers -> treat as pod
-    pack(data, buffer, position, comm, std::integral_constant<bool,true>());
-}
-
 void pack(const data::Well& data, std::vector<char>& buffer, int& position,
           Dune::MPIHelper::MPICommunicator comm)
 {
@@ -689,27 +670,6 @@ void unpack(std::unordered_map<T1,T2,H,P,A>& data, std::vector<char>& buffer, in
     }
 }
 
-void unpack(data::Rates& data, std::vector<char>& buffer, int& position,
-            Dune::MPIHelper::MPICommunicator comm)
-{
-    // plain struct no pointers -> treat as pod
-    unpack(data, buffer, position, comm, std::integral_constant<bool,true>());
-}
-
-void unpack(data::Connection& data, std::vector<char>& buffer, int& position,
-            Dune::MPIHelper::MPICommunicator comm)
-{
-    // plain struct no pointers -> treat as pod
-    unpack(data, buffer, position, comm, std::integral_constant<bool,true>());
-}
-
-void unpack(data::Segment& data,std::vector<char>& buffer, int& position,
-            Dune::MPIHelper::MPICommunicator comm)
-{
-    // plain struct no pointers -> treat as pod
-    unpack(data, buffer, position, comm, std::integral_constant<bool,true>());
-}
-
 void unpack(data::Well& data, std::vector<char>& buffer, int& position,
             Dune::MPIHelper::MPICommunicator comm)
 {