summaryrefslogtreecommitdiff
path: root/lib/worker.h
diff options
context:
space:
mode:
authorDan Goodliffe <dan@randomdan.homeip.net>2023-04-14 11:54:45 +0100
committerDan Goodliffe <dan@randomdan.homeip.net>2023-04-14 11:54:45 +0100
commitaa77548bd637ffd1a7461136e6206906dc4c61c7 (patch)
treed072bbb555685c1eaad250fdd7a02b7a918e3b24 /lib/worker.h
parentSimplify worker with jthread (diff)
downloadilt-aa77548bd637ffd1a7461136e6206906dc4c61c7.tar.bz2
ilt-aa77548bd637ffd1a7461136e6206906dc4c61c7.tar.xz
ilt-aa77548bd637ffd1a7461136e6206906dc4c61c7.zip
New WorkItem/job/promise/future based interface
Diffstat (limited to 'lib/worker.h')
-rw-r--r--lib/worker.h87
1 files changed, 77 insertions, 10 deletions
diff --git a/lib/worker.h b/lib/worker.h
index 96593d9..5356606 100644
--- a/lib/worker.h
+++ b/lib/worker.h
@@ -1,6 +1,8 @@
#pragma once
#include <deque>
+#include <functional>
+#include <future>
#include <memory>
#include <mutex>
#include <semaphore>
@@ -9,28 +11,93 @@
#include <utility>
#include <vector>
-class Work;
class Worker {
+public:
+ class WorkItem {
+ public:
+ WorkItem() = default;
+ virtual ~WorkItem() = default;
+ NO_MOVE(WorkItem);
+ NO_COPY(WorkItem);
+
+ virtual void doWork() = 0;
+ };
+
+ template<typename T> class WorkItemT : public WorkItem {
+ public:
+ T
+ get()
+ {
+ return future.get();
+ }
+
+ protected:
+ std::promise<T> promise;
+ std::future<T> future {promise.get_future()};
+ friend Worker;
+ };
+
+ template<typename... Params>
+ static auto
+ addWork(Params &&... params)
+ {
+ return instance.addWorkImpl(std::forward<Params>(params)...);
+ }
+ template<typename T> using WorkPtrT = std::shared_ptr<WorkItemT<T>>;
+
private:
+ template<typename T, typename... Params> class WorkItemTImpl : public WorkItemT<T> {
+ public:
+ WorkItemTImpl(Params &&... params) : params {std::forward<Params>(params)...} { }
+
+ private:
+ void
+ doWork() override
+ {
+ try {
+ if constexpr (std::is_void_v<T>) {
+ std::apply(
+ [](auto &&... p) {
+ return std::invoke(p...);
+ },
+ params);
+ WorkItemT<T>::promise.set_value();
+ }
+ else {
+ WorkItemT<T>::promise.set_value(std::apply(
+ [](auto &&... p) {
+ return std::invoke(p...);
+ },
+ params));
+ }
+ }
+ catch (...) {
+ WorkItemT<T>::promise.set_exception(std::current_exception());
+ }
+ }
+
+ std::tuple<Params...> params;
+ };
+
Worker();
~Worker();
NO_COPY(Worker);
NO_MOVE(Worker);
- using WorkPtr = std::unique_ptr<Work>;
+ using WorkPtr = std::shared_ptr<WorkItem>;
- template<typename T, typename... Params>
- void
- addWork(Params &&... params)
- requires std::is_base_of_v<Work, T>
+ template<typename... Params>
+ auto
+ addWorkImpl(Params &&... params)
{
- addWork(std::make_unique<T>(std::forward<Params>(params)...));
+ using T = decltype(std::invoke(std::forward<Params>(params)...));
+ auto work = std::make_shared<WorkItemTImpl<T, Params...>>(std::forward<Params>(params)...);
+ addWorkPtr(work);
+ return work;
}
- void addWork(WorkPtr w);
-
-private:
+ void addWorkPtr(WorkPtr w);
void worker();
using Threads = std::vector<std::jthread>;