From de568b32b85b24fb24fdec2bc90066bf00b2f013 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Petr=20Mr=C3=A1zek?= <peterix@gmail.com>
Date: Mon, 10 Dec 2018 20:50:15 +0100
Subject: NOISSUE model Task states as one enum instead of multiple flags

This adds Task::State::AbortedByUser as a possibility
---
 api/logic/tasks/Task.h | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

(limited to 'api/logic/tasks/Task.h')

diff --git a/api/logic/tasks/Task.h b/api/logic/tasks/Task.h
index 58dbc4ca..c66622aa 100644
--- a/api/logic/tasks/Task.h
+++ b/api/logic/tasks/Task.h
@@ -24,6 +24,16 @@
 class MULTIMC_LOGIC_EXPORT Task : public QObject
 {
     Q_OBJECT
+public:
+    enum class State
+    {
+        Inactive,
+        Running,
+        Succeeded,
+        Failed,
+        AbortedByUser
+    };
+
 public:
     explicit Task(QObject *parent = 0);
     virtual ~Task() {};
@@ -88,9 +98,7 @@ public slots:
     void setProgress(qint64 current, qint64 total);
 
 private:
-    bool m_running = false;
-    bool m_finished = false;
-    bool m_succeeded = false;
+    State m_state = State::Inactive;
     QStringList m_Warnings;
     QString m_failReason = "";
     QString m_status;
-- 
cgit v1.2.3