From: Martin Quinson <martin.quinson@loria.fr>
Date: Thu, 6 Jul 2017 08:54:06 +0000 (+0200)
Subject: s4u::Actor::execute() dont need to return a state
X-Git-Tag: v3_17~442
X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/commitdiff_plain/5b471fd312578823ddf19badd84c523c024e99bf?hp=7ef49c428ab0209965a09a36ab28b59789aaa4b5

s4u::Actor::execute() dont need to return a state

It could be used to say if the host failed during that execution, but
the actor would be killed on such failure anyway. So it can only
return "no problem". Not that useful.
---

diff --git a/include/simgrid/s4u/Actor.hpp b/include/simgrid/s4u/Actor.hpp
index 960118c128..4d948964e9 100644
--- a/include/simgrid/s4u/Actor.hpp
+++ b/include/simgrid/s4u/Actor.hpp
@@ -300,7 +300,7 @@ template <class Rep, class Period> inline void sleep_for(std::chrono::duration<R
   }
 
   /** Block the actor, computing the given amount of flops */
-  XBT_PUBLIC(e_smx_state_t) execute(double flop);
+  XBT_PUBLIC(void) execute(double flop);
 
   /** Block the actor until it gets a message from the given mailbox.
    *
diff --git a/src/s4u/s4u_actor.cpp b/src/s4u/s4u_actor.cpp
index b8c5c27a66..3d8a38ed30 100644
--- a/src/s4u/s4u_actor.cpp
+++ b/src/s4u/s4u_actor.cpp
@@ -216,9 +216,10 @@ XBT_PUBLIC(void) sleep_until(double timeout)
     simcall_process_sleep(timeout - now);
 }
 
-e_smx_state_t execute(double flops) {
+void execute(double flops)
+{
   smx_activity_t s = simcall_execution_start(nullptr,flops,1.0/*priority*/,0./*bound*/);
-  return simcall_execution_wait(s);
+  simcall_execution_wait(s);
 }
 
 void* recv(MailboxPtr chan) {