Make JobResult inheritable and remove obsolete Status enum.

src/libcalamares/Job.h

@@ -29,37 +29,33 @@ namespace Calamares {
 class DLLEXPORT JobResult
 {
 public:
-    operator bool() const;
+    virtual ~JobResult() {}
 
-    QString message() const;
-    void setMessage( const QString& message );
+    virtual operator bool() const;
 
-    QString details() const;
-    void setDetails( const QString& details );
+    virtual QString message() const;
+    virtual void setMessage( const QString& message );
+
+    virtual QString details() const;
+    virtual void setDetails( const QString& details );
 
     static JobResult ok();
 
     static JobResult error( const QString& message, const QString& details = QString() );
 
+protected:
+    explicit JobResult( bool ok, const QString& message, const QString& details );
+
 private:
     bool m_ok;
     QString m_message;
     QString m_details;
-
-    JobResult( bool ok, const QString& message, const QString& details );
 };
 
 class DLLEXPORT Job : public QObject
 {
     Q_OBJECT
 public:
-    enum State
-    {
-        Pending = 0,
-        Running,
-        Finished
-    };
-
     explicit Job( QObject* parent = nullptr );
     virtual ~Job();