Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
We never use the name of the mailbox
[simgrid.git] / src / s4u / s4u_mailbox.cpp
index 14f59b4..3b8c0ba 100644 (file)
@@ -18,24 +18,20 @@ using namespace simgrid;
 boost::unordered_map <std::string, s4u::Mailbox *> *s4u::Mailbox::mailboxes = new boost::unordered_map<std::string, s4u::Mailbox*> ();
 
 
-s4u::Mailbox::Mailbox(const char*name, smx_rdv_t inferior) {
+s4u::Mailbox::Mailbox(const char*name, smx_mailbox_t inferior) {
   inferior_ = inferior;
-  name_ = name;
   mailboxes->insert({name, this});
 }
-const char *s4u::Mailbox::getName() {
-  return name_.c_str();
-}
 s4u::Mailbox *s4u::Mailbox::byName(const char*name) {
   s4u::Mailbox *res;
   try {
     res = mailboxes->at(name);
   } catch (std::out_of_range& e) {
     // FIXME: there is a potential race condition here where two actors run Mailbox::byName on a non-existent mailbox
-    // during the same scheduling round. Both will be interrupted in the simcall creating the underlying simix rdv.
+    // during the same scheduling round. Both will be interrupted in the simcall creating the underlying simix mbox.
     // Only one simix object will be created, but two S4U objects will be created.
     // Only one S4U object will be stored in the hashmap and used, and the other one will be leaked.
-    new Mailbox(name,simcall_rdv_create(name));
+    new Mailbox(name,simcall_mbox_create(name));
     res = mailboxes->at(name); // Use the stored one, even if it's not the one I created myself.
   }
   return res;