From 4ae35b504e365c36f9131c28e6b5b4d73ebce1b4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Aum=C3=BCller?= Date: Sun, 11 May 2025 21:57:01 +0200 Subject: [PATCH] Revert "Make sure we free the Boost.MPI type built for C++ bool (sice there is no MPI Bool type)" MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This reverts commit c5270b066f6f810b2fa184650a4215bb5c83ba27. Since #163 "Associate primitive bool type with MPI_CXX_BOOL and make it a logical… " was merged, a dedicated data type for MPI Bool is no longer registered. So it should no longer be freed, either. Otherwise, crashes would occur during MPI_Finalize. --- src/mpi_datatype_cache.cpp | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/mpi_datatype_cache.cpp b/src/mpi_datatype_cache.cpp index 1f8607ff..8437e9d0 100644 --- a/src/mpi_datatype_cache.cpp +++ b/src/mpi_datatype_cache.cpp @@ -8,7 +8,6 @@ #include #include -#include #include namespace boost { namespace mpi { namespace detail { @@ -35,9 +34,6 @@ namespace boost { namespace mpi { namespace detail { // ignore errors in the destructor for (stored_map_type::iterator it=impl->map.begin(); it != impl->map.end(); ++it) MPI_Type_free(&(it->second)); - // We explicitly created this one, as there is no equivalent in the MPI standard - MPI_Datatype bool_type = get_mpi_datatype(bool()); - MPI_Type_free(&bool_type); } }