From 54d56a74aaa8d09c801a1a73989a1d5d95a72795 Mon Sep 17 00:00:00 2001 From: Berthold Stoeger Date: Thu, 8 Feb 2018 22:19:03 +0100 Subject: core/imagedownloader.cpp: Make the manager object local to load() The QNetworkAccessManager is only used in the load() function. No point in it being a subobject. Signed-off-by: Berthold Stoeger --- core/imagedownloader.cpp | 1 + core/imagedownloader.h | 1 - 2 files changed, 1 insertion(+), 1 deletion(-) (limited to 'core') diff --git a/core/imagedownloader.cpp b/core/imagedownloader.cpp index f5bc734b2..82ca53026 100644 --- a/core/imagedownloader.cpp +++ b/core/imagedownloader.cpp @@ -34,6 +34,7 @@ void ImageDownloader::load(bool fromHash){ url = QUrl::fromUserInput(QString(picture->filename)); if (url.isValid()) { QEventLoop loop; + QNetworkAccessManager manager; QNetworkRequest request(url); connect(&manager, SIGNAL(finished(QNetworkReply *)), this, SLOT(saveImage(QNetworkReply *))); QNetworkReply *reply = manager.get(request); diff --git a/core/imagedownloader.h b/core/imagedownloader.h index c03a0f565..295b664f4 100644 --- a/core/imagedownloader.h +++ b/core/imagedownloader.h @@ -15,7 +15,6 @@ public: private: struct picture *picture; - QNetworkAccessManager manager; bool loadFromHash; private slots: -- cgit v1.2.3-70-g09d2