ct

Fastcode · Aug 9, 2024 · dba56bc · dba56bc
1 parent 358bb52
commit dba56bc
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/threading/scheduler/Group.cpp b/src/threading/scheduler/Group.cpp
@@ -35,11 +35,11 @@ namespace threading {
         Group::LockHandle::LockHandle(const NUClear::id_t& task_id,
                                       const int& priority,
                                       const bool& locked,
-                                      const std::function<void()>& notify)
-            : task_id(task_id), priority(priority), locked(locked), notify(notify) {}
+                                      std::function<void()> notify)
+            : task_id(task_id), priority(priority), locked(locked), notify(std::move(notify)) {}
 
-        Group::GroupLock::GroupLock(Group& group, const std::shared_ptr<LockHandle>& handle)
-            : group(group), handle(handle) {}
+        Group::GroupLock::GroupLock(Group& group, std::shared_ptr<LockHandle> handle)
+            : group(group), handle(std::move(handle)) {}
 
         Group::GroupLock::~GroupLock() {
             // The notify targets may be trying to lock the group

diff --git a/src/threading/scheduler/Group.hpp b/src/threading/scheduler/Group.hpp
@@ -53,7 +53,7 @@ namespace threading {
                 LockHandle(const NUClear::id_t& task_id,
                            const int& priority,
                            const bool& locked,
-                           const std::function<void()>& notify);
+                           std::function<void()> notify);
 
                 /**
                  * Compare two lock handles by comparing their priority and task id