Avoid unsafe things

[simgrid.git] / src / kernel / context / Context.cpp

diff --git a/src/kernel/context/Context.cpp b/src/kernel/context/Context.cpp
index 29393ea..2288faa 100644 (file)
--- a/src/kernel/context/Context.cpp
+++ b/src/kernel/context/Context.cpp
@@ -17,12 +17,6 @@
 #include "src/kernel/context/Context.hpp"
 #include "src/simix/smx_private.h"
 
-void SIMIX_process_set_cleanup_function(
-  smx_process_t process, void_pfn_smxprocess_t cleanup)
-{
-  process->context->set_cleanup(cleanup);
-}
-
 /**
  * @brief creates a new context for a user level process
  * @param code a main function
@@ -31,10 +25,9 @@ void SIMIX_process_set_cleanup_function(
 smx_context_t SIMIX_context_new(
   std::function<void()> code,
   void_pfn_smxprocess_t cleanup_func,
-  smx_process_t simix_process)
+  smx_actor_t simix_process)
 {
-  if (!simix_global)
-    xbt_die("simix is not initialized, please call MSG_init first");
+  xbt_assert(simix_global, "simix is not initialized, please call MSG_init first");
   return simix_global->context_factory->create_context(
     std::move(code), cleanup_func, simix_process);
 }
@@ -45,7 +38,7 @@ namespace context {
 
 ContextFactoryInitializer factory_initializer = nullptr;
 
-ContextFactory::~ContextFactory() {}
+ContextFactory::~ContextFactory() = default;
 
 Context* ContextFactory::self()
 {
@@ -61,20 +54,20 @@ void ContextFactory::declare_context(void* context, std::size_t size)
 #endif
 }
 
-Context* ContextFactory::attach(void_pfn_smxprocess_t cleanup_func, smx_process_t process)
+Context* ContextFactory::attach(void_pfn_smxprocess_t cleanup_func, smx_actor_t process)
 {
   xbt_die("Cannot attach with this ContextFactory.\n"
     "Try using --cfg=contexts/factory:thread instead.\n");
 }
 
-Context* ContextFactory::create_maestro(std::function<void()> code, smx_process_t process)
+Context* ContextFactory::create_maestro(std::function<void()> code, smx_actor_t process)
 {
   xbt_die("Cannot create_maestro with this ContextFactory.\n"
     "Try using --cfg=contexts/factory:thread instead.\n");
 }
 
 Context::Context(std::function<void()> code,
-    void_pfn_smxprocess_t cleanup_func, smx_process_t process)
+    void_pfn_smxprocess_t cleanup_func, smx_actor_t process)
   : code_(std::move(code)), process_(process), iwannadie(false)
 {
   /* If the user provided a function for the process then use it.
@@ -86,9 +79,7 @@ Context::Context(std::function<void()> code,
     SIMIX_context_set_current(this);
 }
 
-Context::~Context()
-{
-}
+Context::~Context() = default;
 
 void Context::stop()
 {
@@ -101,21 +92,18 @@ void Context::stop()
   this->iwannadie = true;
 }
 
-AttachContext::~AttachContext()
-{
-}
+AttachContext::~AttachContext() = default;
 
 }}}
 
 /** @brief Executes all the processes to run (in parallel if possible). */
-void SIMIX_context_runall(void)
+void SIMIX_context_runall()
 {
-  if (!xbt_dynar_is_empty(simix_global->process_to_run))
-    simix_global->context_factory->run_all();
+  simix_global->context_factory->run_all();
 }
 
 /** @brief returns the current running context */
-smx_context_t SIMIX_context_self(void)
+smx_context_t SIMIX_context_self()
 {
   if (simix_global && simix_global->context_factory)
     return simix_global->context_factory->self();