From 5f85a2bfc5d5fee00fcdfb1230af32344376e39a Mon Sep 17 00:00:00 2001 From: Kitsune Ral Date: Sat, 19 Aug 2017 18:01:19 +0900 Subject: BaseJob::Data: Small update to better match Qt API Also: Query and Data constructors from initialization_list<> are no more explicit, as clang-tidy recommends. --- jobs/basejob.h | 7 +++---- jobs/passwordlogin.cpp | 2 +- 2 files changed, 4 insertions(+), 5 deletions(-) (limited to 'jobs') diff --git a/jobs/basejob.h b/jobs/basejob.h index 0ec40a7a..5df03b32 100644 --- a/jobs/basejob.h +++ b/jobs/basejob.h @@ -63,7 +63,7 @@ namespace QMatrixClient public: using QUrlQuery::QUrlQuery; Query() = default; - explicit Query(const std::initializer_list< QPair >& l) + Query(const std::initializer_list< QPair >& l) { setQueryItems(l); } @@ -78,11 +78,10 @@ namespace QMatrixClient { public: Data() = default; - Data(const QJsonObject& o) : QJsonObject(o) { } - Data(QJsonObject&& o) : QJsonObject(std::move(o)) { } + explicit Data(const QJsonObject& o) : QJsonObject(o) { } #if (QT_VERSION < QT_VERSION_CHECK(5, 4, 0)) // This method exists in QJsonObject of newer Qt versions - explicit Data(const std::initializer_list< QPair >& l) + Data(const std::initializer_list< QPair >& l) { for (auto i: l) insert(i.first, i.second); diff --git a/jobs/passwordlogin.cpp b/jobs/passwordlogin.cpp index 081e19bc..09108215 100644 --- a/jobs/passwordlogin.cpp +++ b/jobs/passwordlogin.cpp @@ -33,7 +33,7 @@ PasswordLogin::PasswordLogin(const ConnectionData* connection, QString user, QSt , "_matrix/client/r0/login" , Query() , Data( - { { "type", "m.login.password" } + { { "type", QStringLiteral("m.login.password") } , { "user", user } , { "password", password } }) -- cgit v1.2.3