Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
improve a comment
authorMartin Quinson <martin.quinson@loria.fr>
Thu, 25 Jan 2018 05:55:39 +0000 (06:55 +0100)
committerMartin Quinson <martin.quinson@loria.fr>
Thu, 25 Jan 2018 05:55:41 +0000 (06:55 +0100)
Shorten this a bit, and do not dupplicate things that we can find with
a git blame.

src/simix/ActorImpl.cpp

index d2b51d4..1fd6dc3 100644 (file)
@@ -351,10 +351,8 @@ smx_actor_t SIMIX_process_create(const char* name, std::function<void()> code, v
 
   /* Tracing the process creation */
   TRACE_msg_process_create(process->getName(), process->pid, process->host);
-  /* Note by cheinrich: If you move this directly after the "new ActorImpl", the pid
-   * will not yet be set and you will cause issues when other code relies on that.
-   * This is of course also true for the other properties, so I moved this here.
-   */
+
+  /* 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::onCreation(tmp);