develop #comment Use nanoseconds as default duration (Scheduler).

 - Avoid useless copies (TimedTask).

Author: Luc Sinet

include/TaskManager/Scheduler.hpp

@@ -15,6 +15,8 @@
 
 namespace Task {
 
+using Duration = std::chrono::nanoseconds;
+
 //! The task scheduler.
 class Scheduler {
  public:
@@ -33,7 +35,7 @@ class Scheduler {
   //! @param delay The delay to wait before executing the task.
   //! @param function The function to execute.
   //! @param args the parameters to pass to the function.
-  template <typename Duration, class F, class... Args>
+  template <class F, class... Args>
   auto scheduleIn(const std::string& id, Duration delay, F&& function, Args&&... args) {
     return this->scheduleAt(id, Detail::Clock::now() + delay, std::forward<F>(function), std::forward<Args>(args)...);
   }
@@ -73,7 +75,7 @@ class Scheduler {
   //! @param delay The minimum duration separating two executions.
   //! @param function The function to execute.
   //! @param args the parameters to pass to the function.
-  template <typename Duration, class F, class... Args>
+  template <class F, class... Args>
   void scheduleEvery(const std::string& id, Duration delay, F&& function, Args&&... args) {
     auto task = std::bind(std::forward<F>(function), std::forward<Args>(args)...);
     const auto hash = _hasher(id);

include/TaskManager/detail/Task.hpp

@@ -3,6 +3,7 @@
 
 #include <chrono>
 #include <functional>
+#include <utility>
 
 namespace Task {
 namespace Detail {
@@ -13,8 +14,8 @@ using Timepoint = Clock::time_point;
 
 class TimedTask {
  public:
-  TimedTask(Task functor, Timepoint timepoint)
-    : _functor(functor)
+  TimedTask(Task functor, const Timepoint& timepoint)
+    : _functor(std::move(functor))
     , _timepoint(timepoint) {}
 
   const Timepoint& timepoint() const {