diff options
author | Kitsune Ral <Kitsune-Ral@users.sf.net> | 2016-07-27 14:54:43 +0900 |
---|---|---|
committer | Kitsune Ral <Kitsune-Ral@users.sf.net> | 2016-07-27 14:54:43 +0900 |
commit | 844d3362022d0278b07e1232460c1662e4fb61d2 (patch) | |
tree | ae00b58eee196782e56839faca52dd8bc3bbd00f /jobs/roommembersjob.h | |
parent | d8c3e31d34b8129dc24f52a7d726ff1206b90d48 (diff) | |
download | libquotient-844d3362022d0278b07e1232460c1662e4fb61d2.tar.gz libquotient-844d3362022d0278b07e1232460c1662e4fb61d2.zip |
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.
Diffstat (limited to 'jobs/roommembersjob.h')
-rw-r--r-- | jobs/roommembersjob.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/jobs/roommembersjob.h b/jobs/roommembersjob.h index ffae4309..04803d67 100644 --- a/jobs/roommembersjob.h +++ b/jobs/roommembersjob.h @@ -35,8 +35,8 @@ namespace QMatrixClient QList<State*> states(); protected: - virtual QString apiPath() const override; - virtual void parseJson(const QJsonDocument& data) override; + QString apiPath() const override; + Status parseJson(const QJsonDocument& data) override; private: class Private; |