use setters

[simgrid.git] / src / kernel / activity / ExecImpl.hpp

diff --git a/src/kernel/activity/ExecImpl.hpp b/src/kernel/activity/ExecImpl.hpp
index 74daa44..b1dc4f7 100644 (file)
--- a/src/kernel/activity/ExecImpl.hpp
+++ b/src/kernel/activity/ExecImpl.hpp
@@ -16,20 +16,21 @@ namespace activity {
 
 class XBT_PUBLIC ExecImpl : public ActivityImpl {
   resource::Action* timeout_detector_ = nullptr;
-  ~ExecImpl() override;
+  ~ExecImpl();
 
 public:
-  explicit ExecImpl(const std::string& name, const std::string& tracing_category);
   ExecImpl* start(double flops_amount, double priority, double bound);
   ExecImpl* start(const std::vector<s4u::Host*>& hosts, const std::vector<double>& flops_amounts,
                   const std::vector<double>& bytes_amounts);
 
+  ExecImpl* set_name(const std::string& name);
+  ExecImpl* set_tracing_category(const std::string& category);
   ExecImpl* set_host(s4u::Host* host);
   ExecImpl* set_timeout(double timeout);
   void cancel();
   void post() override;
   void finish() override;
-  double get_remaining();
+  double get_remaining() const;
   double get_seq_remaining_ratio();
   double get_par_remaining_ratio();
   void set_bound(double bound);       // deprecated. To be removed in v3.25
@@ -39,9 +40,9 @@ public:
   /* The host where the execution takes place. nullptr means this is a parallel exec (and only surf knows the hosts) */
   s4u::Host* host_ = nullptr;
 
-  static xbt::signal<void(ExecImplPtr)> on_creation;
-  static xbt::signal<void(ExecImplPtr)> on_completion;
-  static xbt::signal<void(ExecImplPtr, s4u::Host*)> on_migration;
+  static xbt::signal<void(ExecImpl&)> on_creation;
+  static xbt::signal<void(ExecImpl const&)> on_completion;
+  static xbt::signal<void(ExecImpl const&, s4u::Host*)> on_migration;
 };
 } // namespace activity
 } // namespace kernel