address comments

diff --git a/include/deal.II/base/mpi.h b/include/deal.II/base/mpi.h
index a5e897bc6f6c593d595de1ec327e381b42d75bab..35e136b5d8eb65d0729c67af7e74953019845b72 100644 (file)
--- a/include/deal.II/base/mpi.h
+++ b/include/deal.II/base/mpi.h
@@ -233,6 +233,11 @@ namespace Utilities
         : comm(duplicate_communicator(communicator))
       {}
 
+      /**
+       * Do not allow making copies.
+       */
+      DuplicatedCommunicator(const DuplicatedCommunicator &) = delete;
+
       /**
        * The destructor will free the communicator automatically.
        */
@@ -249,10 +254,6 @@ namespace Utilities
         return comm;
       }
 
-      /**
-       * Do not allow making copies.
-       */
-      DuplicatedCommunicator(const DuplicatedCommunicator &) = delete;
 
       /**
        * Do not allow assignment of this class.

diff --git a/source/base/mpi.cc b/source/base/mpi.cc
index 3c7318b5191194a36edf7a096bf7b08c11d284b7..32bde89b6c1df80a1f17a9e51e1a427e5d7f9aa8 100644 (file)
--- a/source/base/mpi.cc
+++ b/source/base/mpi.cc
@@ -106,6 +106,7 @@ namespace Utilities
     void
     free_communicator(MPI_Comm &mpi_communicator)
     {
+      // MPI_Comm_free will set the argument to MPI_COMM_NULL automatically.
       const int ierr = MPI_Comm_free(&mpi_communicator);
       AssertThrowMPI(ierr);
     }