Pass large parameter by const reference.

diff --git a/src/smpi/include/smpi_group.hpp b/src/smpi/include/smpi_group.hpp
index 1ee962b..db5e93d 100644 (file)
--- a/src/smpi/include/smpi_group.hpp
+++ b/src/smpi/include/smpi_group.hpp
@@ -25,8 +25,8 @@ class Group : public F2C{
 
   int refcount_ = 1; /* refcount_: start > 0 so that this group never gets freed */
 
-  int incl(const std::vector<int> ranks, MPI_Group* newgroup) const;
-  int excl(const std::vector<bool> excl_map, MPI_Group* newgroup) const;
+  int incl(const std::vector<int>& ranks, MPI_Group* newgroup) const;
+  int excl(const std::vector<bool>& excl_map, MPI_Group* newgroup) const;
 
 public:
   Group() = default;

diff --git a/src/smpi/mpi/smpi_group.cpp b/src/smpi/mpi/smpi_group.cpp
index e97d8da..3e32408 100644 (file)
--- a/src/smpi/mpi/smpi_group.cpp
+++ b/src/smpi/mpi/smpi_group.cpp
@@ -116,12 +116,12 @@ int Group::incl(int n, const int* ranks, MPI_Group* newgroup) const
   return MPI_SUCCESS;
 }
 
-int Group::incl(const std::vector<int> ranks, MPI_Group* newgroup) const
+int Group::incl(const std::vector<int>& ranks, MPI_Group* newgroup) const
 {
   return incl(static_cast<int>(ranks.size()), ranks.data(), newgroup);
 }
 
-int Group::excl(const std::vector<bool> excl_map, MPI_Group* newgroup) const
+int Group::excl(const std::vector<bool>& excl_map, MPI_Group* newgroup) const
 {
   xbt_assert(static_cast<int>(excl_map.size()) == size());
   std::vector<int> ranks;