OilfieldToolsNet/opm-simulators
4
0
Fork 0
mirror of https://github.com/OPM/opm-simulators.git synced 2025-02-25 18:55:30 -06:00
Code Issues Packages Projects Releases Wiki Activity

MPIPacker: mark buffer const in unpacker functions

Browse Source
This commit is contained in:
Arne Morten Kvarving
2024-09-11 12:46:13 +02:00
parent e808a3d2f3
commit e559fc371b
2 changed files with 12 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
opm/simulators/utils/MPIPacker.cpp
View File

@@ -63,7 +63,7 @@ pack(const std::bitset<Size>& data,
template<std::size_t Size>
void Packing<false,std::bitset<Size>>::
unpack(std::bitset<Size>& data,
std::vector<char>& buffer,
const std::vector<char>& buffer,
std::size_t& position,
Parallel::MPIComm comm)
{
@@ -99,7 +99,7 @@ pack(const std::string& data,
void Packing<false,std::string>::
unpack(std::string& data,
std::vector<char>& buffer,
const std::vector<char>& buffer,
std::size_t& position,
Opm::Parallel::MPIComm comm)
{
@@ -133,7 +133,7 @@ pack(const time_point& data,
void Packing<false,time_point>::
unpack(time_point& data,
std::vector<char>& buffer,
const std::vector<char>& buffer,
std::size_t& position,
Parallel::MPIComm comm)
{

17
opm/simulators/utils/MPIPacker.hpp
View File

@@ -42,7 +42,7 @@ struct Packing
{
static std::size_t packSize(const T&, Parallel::MPIComm);
static void pack(const T&, std::vector<char>&, std::size_t&, Parallel::MPIComm);
static void unpack(T&, std::vector<char>&, std::size_t&, Parallel::MPIComm);
static void unpack(T&, const std::vector<char>&, std::size_t&, Parallel::MPIComm);
};
//! \brief Packaging for pod data.
@@ -109,7 +109,7 @@ struct Packing<true,T>
//! \param position Position in buffer to use
//! \param comm The communicator to use
static void unpack(T& data,
std::vector<char>& buffer,
const std::vector<char>& buffer,
std::size_t& position,
Parallel::MPIComm comm)
{
@@ -124,7 +124,7 @@ struct Packing<true,T>
//! \param comm The communicator to use
static void unpack(T* data,
std::size_t n,
std::vector<char>& buffer,
const std::vector<char>& buffer,
std::size_t& position,
Parallel::MPIComm comm)
{
@@ -151,7 +151,7 @@ struct Packing<false,T>
static_assert(!std::is_same_v<T,T>, "Packing not supported for type");
}
static void unpack(T&, std::vector<char>&, std::size_t&,
static void unpack(T&, const std::vector<char>&, std::size_t&,
Parallel::MPIComm)
{
static_assert(!std::is_same_v<T,T>, "Packing not supported for type");
@@ -164,7 +164,8 @@ struct Packing<false,std::bitset<Size>>
{
static std::size_t packSize(const std::bitset<Size>&, Opm::Parallel::MPIComm);
static void pack(const std::bitset<Size>&, std::vector<char>&, std::size_t&, Opm::Parallel::MPIComm);
static void unpack(std::bitset<Size>&, std::vector<char>&, std::size_t&, Opm::Parallel::MPIComm);
static void unpack(std::bitset<Size>&, const std::vector<char>&,
std::size_t&, Opm::Parallel::MPIComm);
};
#define ADD_PACK_SPECIALIZATION(T) \
@@ -173,7 +174,7 @@ struct Packing<false,std::bitset<Size>>
{ \
static std::size_t packSize(const T&, Parallel::MPIComm); \
static void pack(const T&, std::vector<char>&, std::size_t&, Parallel::MPIComm); \
static void unpack(T&, std::vector<char>&, std::size_t&, Parallel::MPIComm); \
static void unpack(T&, const std::vector<char>&, std::size_t&, Parallel::MPIComm); \
};
ADD_PACK_SPECIALIZATION(std::string)
@@ -248,7 +249,7 @@ struct Packer
//! \param position Position in buffer to use
template<class T>
void unpack(T& data,
std::vector<char>& buffer,
const std::vector<char>& buffer,
std::size_t& position) const
{
detail::Packing<std::is_pod_v<T>,T>::unpack(data, buffer, position, m_comm);
@@ -263,7 +264,7 @@ struct Packer
template<class T>
void unpack(T* data,
std::size_t n,
std::vector<char>& buffer,
const std::vector<char>& buffer,
std::size_t& position) const
{
static_assert(std::is_pod_v<T>, "Array packing not supported for non-pod data");