Further tweaks to MxcReply

- QNetworkReply::isSequential() already returns `true`, there's no need
to overload it again.
- Use `Q_SLOTS` instead of `slots` because it's an external library
  interface and clients may use other libraries using `slots` identifier;
- Use `emit` instead of `Q_EMIT` because this is a part of internal
  implementation and if we ever use a library that has an `emit`
  identifier, a massive search-replace will be in order anyway.
- Use `QMetaObject::invokeMethod()` with a queued connection as
  a clearer way to achieve the same goal as `QTimer::singleShot(0, ...)`.

1 files changed, 7 insertions, 8 deletions

diff --git a/lib/mxcreply.h b/lib/mxcreply.h
index ac3ac4f4..efaf01c6 100644
--- a/lib/mxcreply.h
+++ b/lib/mxcreply.h
@@ -8,23 +8,22 @@
 
 namespace Quotient {
 class Room;
-class Connection;
+
 class MxcReply : public QNetworkReply
 {
 public:
-    MxcReply(QNetworkReply* reply, Room* room, const QString &eventId);
-    MxcReply(QNetworkReply* reply);
-    MxcReply();
-
-    bool isSequential() const override;
+    explicit MxcReply();
+    explicit MxcReply(QNetworkReply *reply);
+    MxcReply(QNetworkReply* reply, Room* room, const QString& eventId);
 
-public slots:
+public Q_SLOTS:
     void abort() override;
 
 protected:
     qint64 readData(char *data, qint64 maxSize) override;
+
 private:
     class Private;
     std::unique_ptr<Private> d;
 };
-}
\ No newline at end of file
+}