LSP: default initialize Request member

and add some missing constructors

Change-Id: I0496de183562d54dc8b4115270622f3760f4ef84
Reviewed-by: Eike Ziller <eike.ziller@qt.io>
This commit is contained in:
David Schulz
2018-09-12 10:18:05 +02:00
parent c530775cb9
commit 82ea862f50

View File

@@ -224,10 +224,12 @@ template <typename Result, typename Error, typename Params>
class Request : public Notification<Params>
{
public:
Request() : Notification<Params>(), m_callBack(0) { setId(QUuid::createUuid().toString()); }
Request() : Notification<Params>() { setId(QUuid::createUuid().toString()); }
Request(const QString &methodName, const Params &params = Params())
: Notification<Params>(methodName, params), m_callBack(0)
: Notification<Params>(methodName, params)
{ setId(QUuid::createUuid().toString()); }
Request(const QJsonObject &jsonObject) : Notification<Params>(jsonObject) { }
Request(QJsonObject &&jsonObject) : Notification<Params>(std::move(jsonObject)) { }
MessageId id() const
{ return MessageId(JsonRpcMessage::m_jsonObject.value(idKey)); }