Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
cleanups around the actor terminaison
authorMartin Quinson <martin.quinson@ens-rennes.fr>
Tue, 15 Jan 2019 20:19:44 +0000 (21:19 +0100)
committerMartin Quinson <martin.quinson@ens-rennes.fr>
Tue, 15 Jan 2019 21:28:20 +0000 (22:28 +0100)
src/kernel/context/Context.cpp
src/simix/ActorImpl.cpp

index c04ae5b..cdce4e0 100644 (file)
@@ -66,14 +66,11 @@ Context* ContextFactory::create_maestro(std::function<void()> code, smx_actor_t
 }
 
 Context::Context(std::function<void()> code, void_pfn_smxprocess_t cleanup_func, smx_actor_t actor)
 }
 
 Context::Context(std::function<void()> code, void_pfn_smxprocess_t cleanup_func, smx_actor_t actor)
-    : code_(std::move(code)), actor_(actor)
+    : code_(std::move(code)), cleanup_func_(cleanup_func), actor_(actor)
 {
 {
-  /* If the user provided a function for the process then use it.
-     Otherwise, it is the context for maestro and we should set it as the
-     current context */
-  if (has_code())
-    this->cleanup_func_ = cleanup_func;
-  else
+  /* If no function was provided, this is the context for maestro
+   * and we should set it as the current context */
+  if (not has_code())
     set_current(this);
 }
 
     set_current(this);
 }
 
index 73c1c7e..06e64af 100644 (file)
@@ -369,9 +369,8 @@ smx_actor_t SIMIX_process_create(std::string name, simgrid::simix::ActorCode cod
   simix_global->process_to_run.push_back(process);
   intrusive_ptr_add_ref(process);
 
   simix_global->process_to_run.push_back(process);
   intrusive_ptr_add_ref(process);
 
-  /* The onCreation() signal must be delayed until there, where the pid and everything is set */
-  simgrid::s4u::ActorPtr tmp = process->iface(); // Passing this directly to onCreation will lead to crashes
-  simgrid::s4u::Actor::on_creation(tmp);
+  /* The on_creation() signal must be delayed until there, where the pid and everything is set */
+  simgrid::s4u::Actor::on_creation(process->iface());
 
   return process;
 }
 
   return process;
 }