Introduce Status class + BaseJob::{checkReply,parseReply,parseJson} now return it

This better fixes the contract for derived job classes and simplifies error reporting. Methods error() and errorString() are kept for back-compatibility; status() returns a combination of them, conveniently packed into a Status object. For a quick status check, Status::good() is provided.

1 files changed, 1 insertions, 1 deletions

diff --git a/jobs/checkauthmethods.h b/jobs/checkauthmethods.h
index 2951e27f..36eaa01a 100644
--- a/jobs/checkauthmethods.h
+++ b/jobs/checkauthmethods.h
@@ -35,7 +35,7 @@ namespace QMatrixClient
             
         protected:
             QString apiPath() const override;
-            void parseJson(const QJsonDocument& data) override;
+            Status parseJson(const QJsonDocument& data) override;
             
         private:
             class Private;