diff options
| author | flow <flowlnlnln@gmail.com> | 2023-01-24 16:52:09 -0300 |
|---|---|---|
| committer | flow <flowlnlnln@gmail.com> | 2023-01-25 16:57:51 -0300 |
| commit | 29f7ea752fd34bdea64a7c7f2c505982ac39ce0d (patch) | |
| tree | f3d76dd640ae4f6a241fb0ff4532d938dc1c33bd /launcher/news/NewsChecker.cpp | |
| parent | 5186ad95d3cd66da8c844d9c3c0cd95b8b2f0b94 (diff) | |
| download | PrismLauncher-29f7ea752fd34bdea64a7c7f2c505982ac39ce0d.tar.gz PrismLauncher-29f7ea752fd34bdea64a7c7f2c505982ac39ce0d.tar.bz2 PrismLauncher-29f7ea752fd34bdea64a7c7f2c505982ac39ce0d.zip | |
refactor: make shared_qobject_ptr ctor explicit
This turns issues like creating two shared ptrs from a single raw ptr
from popping up at runtime, instead making them a compile error.
Signed-off-by: flow <flowlnlnln@gmail.com>
Diffstat (limited to 'launcher/news/NewsChecker.cpp')
| -rw-r--r-- | launcher/news/NewsChecker.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/launcher/news/NewsChecker.cpp b/launcher/news/NewsChecker.cpp index 3b969732..1f1520d0 100644 --- a/launcher/news/NewsChecker.cpp +++ b/launcher/news/NewsChecker.cpp @@ -57,10 +57,10 @@ void NewsChecker::reloadNews() qDebug() << "Reloading news."; - NetJob* job = new NetJob("News RSS Feed", m_network); + NetJob::Ptr job{ new NetJob("News RSS Feed", m_network) }; job->addNetAction(Net::Download::makeByteArray(m_feedUrl, &newsData)); - QObject::connect(job, &NetJob::succeeded, this, &NewsChecker::rssDownloadFinished); - QObject::connect(job, &NetJob::failed, this, &NewsChecker::rssDownloadFailed); + QObject::connect(job.get(), &NetJob::succeeded, this, &NewsChecker::rssDownloadFinished); + QObject::connect(job.get(), &NetJob::failed, this, &NewsChecker::rssDownloadFailed); m_newsNetJob.reset(job); job->start(); } |