Don't use pass-by-value for large parameters.

[simgrid.git] / src / msg / msg_private.hpp

diff --git a/src/msg/msg_private.hpp b/src/msg/msg_private.hpp
index 652df4e..ab76546 100644 (file)
--- a/src/msg/msg_private.hpp
+++ b/src/msg/msg_private.hpp
@@ -29,8 +29,8 @@ class Task {
   bool is_used_    = false; /* Indicates whether the task is used in SIMIX currently */
 
   explicit Task(std::string name, double flops_amount, double bytes_amount, void* data);
-  explicit Task(std::string name, std::vector<s4u::Host*> hosts, std::vector<double> flops_amount,
-                std::vector<double> bytes_amount, void* data);
+  explicit Task(std::string name, std::vector<s4u::Host*>&& hosts, std::vector<double>&& flops_amount,
+                std::vector<double>&& bytes_amount, void* data);
 
   void report_multiple_use() const;
 
@@ -99,14 +99,10 @@ public:
 
 class ActorUserData {
   void* userdata_      = nullptr;
-  s4u::ActorPtr actor_ = nullptr;
 
 public:
   static xbt::Extension<simgrid::s4u::Actor, ActorUserData> EXTENSION_ID;
 
-  explicit ActorUserData(s4u::ActorPtr ptr) : actor_(ptr) {}
-  ~ActorUserData() = default;
-
   void set_user_data(void* data) { userdata_ = data; }
   void* get_user_data() { return userdata_; }
 };