Use default member initializers and constructor delegation

[builder.git] / source / internaltask.h

diff --git a/source/internaltask.h b/source/internaltask.h
index ea25ba89eca0816751d8f13687b5e4aa5afeefd8..98b76a0bb3a904391ae4643a76104daebf4cbc25 100644 (file)
--- a/source/internaltask.h
+++ b/source/internaltask.h
@@ -4,15 +4,22 @@
 #include <msp/core/thread.h>
 #include "task.h"
 
+/**
+Runs a worker thread.  Tools should derive a thread class from
+InternalTask::Worker.  The worker thread must set its status to either SUCCESS
+or ERROR before terminating.
+*/
 class InternalTask: public Task
 {
 public:
        class Worker: public Msp::Thread
        {
+               friend class InternalTask;
+
        protected:
-               Status status;
+               volatile Status status = Task::RUNNING;
 
-               Worker();
+               Worker() = default;
 
        public:
                Status get_status() const { return status; }
@@ -25,7 +32,10 @@ public:
        InternalTask(Worker *);
        ~InternalTask();
 
-       virtual Status check();
+       std::string get_command() const override { return "<internal>"; }
+       void start() override;
+       Status check() override;
+       Status wait() override;
 };
 
 #endif