Use a std::unique_ptr, and avoid explicit new/delete.

[simgrid.git] / src / smpi / include / smpi_group.hpp

diff --git a/src/smpi/include/smpi_group.hpp b/src/smpi/include/smpi_group.hpp
index d653812..1cec217 100644 (file)
--- a/src/smpi/include/smpi_group.hpp
+++ b/src/smpi/include/smpi_group.hpp
@@ -1,4 +1,4 @@
-/* Copyright (c) 2010-2021. The SimGrid Team.
+/* Copyright (c) 2010-2022. The SimGrid Team.
  * All rights reserved.                                                     */
 
 /* This program is free software; you can redistribute it and/or modify it
@@ -16,10 +16,9 @@ namespace simgrid{
 namespace smpi{
 
 class Group : public F2C{
-  /* This is actually a map from int to s4u::Actor*. We could use std::map here, but looking up a value there costs
+  /* This is actually a map from int to aid_t. We could use std::map here, but looking up a value there costs
    * O(log(n)). For a vector, this costs O(1). We hence go with the vector.
    */
-  std::map<s4u::Actor*, int> actor_to_rank_map_;
   std::vector<aid_t> rank_to_pid_map_;
   std::vector<int> pid_to_rank_map_;
 
@@ -35,9 +34,8 @@ public:
 
   void set_mapping(aid_t pid, int rank);
   int rank(aid_t pid) const;
-  aid_t actor_pid(int rank) const;
+  aid_t actor(int rank) const;
   std::string name() const override {return std::string("MPI_Group");}
-  int rank(s4u::Actor* process) const;
   void ref();
   static void unref(MPI_Group group);
   int size() const { return static_cast<int>(rank_to_pid_map_.size()); }