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/java/JavaInstallList.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/java/JavaInstallList.cpp')
| -rw-r--r-- | launcher/java/JavaInstallList.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/launcher/java/JavaInstallList.cpp b/launcher/java/JavaInstallList.cpp index e2f0aa00..b29af857 100644 --- a/launcher/java/JavaInstallList.cpp +++ b/launcher/java/JavaInstallList.cpp @@ -67,7 +67,7 @@ void JavaInstallList::load() if(m_status != Status::InProgress) { m_status = Status::InProgress; - m_loadTask = new JavaListLoadTask(this); + m_loadTask.reset(new JavaListLoadTask(this)); m_loadTask->start(); } } @@ -167,7 +167,7 @@ void JavaListLoadTask::executeTask() JavaUtils ju; QList<QString> candidate_paths = ju.FindJavaPaths(); - m_job = new JavaCheckerJob("Java detection"); + m_job.reset(new JavaCheckerJob("Java detection")); connect(m_job.get(), &Task::finished, this, &JavaListLoadTask::javaCheckerFinished); connect(m_job.get(), &Task::progress, this, &Task::setProgress); |