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/joinroomjob.h b/jobs/joinroomjob.h
index 1329ca56..a6f4af21 100644
--- a/jobs/joinroomjob.h
+++ b/jobs/joinroomjob.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;