Mirrors/PrismLauncher
1
0
Fork 0
mirror of https://github.com/PrismLauncher/PrismLauncher.git synced 2025-05-30 13:50:20 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix: 6.2 deprecation warning regard the QScopedPointer::swap function

Browse Source 
Signed-off-by: Trial97 <alexandru.tripon97@gmail.com>
This commit is contained in:
Trial97 2025-04-18 13:15:00 +03:00
parent 3df98dd5e7
commit e3ff9630e9
No known key found for this signature in database
GPG Key ID: 55EF5DA53DB36318
12 changed files with 36 additions and 36 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
launcher/ui/pages/modplatform/ModPage.cpp
View File

@ -62,22 +62,24 @@ ModPage::ModPage(ModDownloadDialog* dialog, BaseInstance& instance) : ResourcePa
connect(m_ui->packView, &QListView::doubleClicked, this, &ModPage::onResourceSelected); connect(m_ui->packView, &QListView::doubleClicked, this, &ModPage::onResourceSelected);
} }
void ModPage::setFilterWidget(unique_qobject_ptr<ModFilterWidget>& widget) void ModPage::setFilterWidget(ModFilterWidget* widget)
{ {
if (m_filter_widget) if (m_filter_widget)
disconnect(m_filter_widget.get(), nullptr, nullptr, nullptr); disconnect(m_filter_widget, nullptr, nullptr, nullptr);
auto old = m_ui->splitter->replaceWidget(0, widget.get()); auto old = m_ui->splitter->replaceWidget(0, widget);
// because we replaced the widget we also need to delete it // because we replaced the widget we also need to delete it
if (old) { if (old) {
delete old; old->deleteLater();
} }
m_filter_widget.swap(widget); m_filter_widget = widget;
if (m_filter_widget) {
m_filter_widget->deleteLater();
}
m_filter = m_filter_widget->getFilter(); m_filter = m_filter_widget->getFilter();
connect(m_filter_widget.get(), &ModFilterWidget::filterChanged, this, &ModPage::triggerSearch); connect(m_filter_widget, &ModFilterWidget::filterChanged, this, &ModPage::triggerSearch);
prepareProviderCategories(); prepareProviderCategories();
} }

6
launcher/ui/pages/modplatform/ModPage.h
View File

@ -50,11 +50,11 @@ class ModPage : public ResourcePage {
void addResourceToPage(ModPlatform::IndexedPack::Ptr, ModPlatform::IndexedVersion&, std::shared_ptr<ResourceFolderModel>) override; void addResourceToPage(ModPlatform::IndexedPack::Ptr, ModPlatform::IndexedVersion&, std::shared_ptr<ResourceFolderModel>) override;
virtual unique_qobject_ptr<ModFilterWidget> createFilterWidget() = 0; virtual ModFilterWidget* createFilterWidget() = 0;
[[nodiscard]] bool supportsFiltering() const override { return true; }; [[nodiscard]] bool supportsFiltering() const override { return true; };
auto getFilter() const -> const std::shared_ptr<ModFilterWidget::Filter> { return m_filter; } auto getFilter() const -> const std::shared_ptr<ModFilterWidget::Filter> { return m_filter; }
void setFilterWidget(unique_qobject_ptr<ModFilterWidget>&); void setFilterWidget(ModFilterWidget*);
protected: protected:
ModPage(ModDownloadDialog* dialog, BaseInstance& instance); ModPage(ModDownloadDialog* dialog, BaseInstance& instance);
@ -66,7 +66,7 @@ class ModPage : public ResourcePage {
void triggerSearch() override; void triggerSearch() override;
protected: protected:
unique_qobject_ptr<ModFilterWidget> m_filter_widget; ModFilterWidget* m_filter_widget = nullptr;
std::shared_ptr<ModFilterWidget::Filter> m_filter; std::shared_ptr<ModFilterWidget::Filter> m_filter;
}; };

13
launcher/ui/pages/modplatform/flame/FlamePage.cpp
View File

@ -341,17 +341,20 @@ void FlamePage::setSearchTerm(QString term)
void FlamePage::createFilterWidget() void FlamePage::createFilterWidget()
{ {
auto widget = ModFilterWidget::create(nullptr, false, this); auto widget = new ModFilterWidget(nullptr, false, this);
m_filterWidget.swap(widget); if (m_filterWidget) {
auto old = ui->splitter->replaceWidget(0, m_filterWidget.get()); m_filterWidget->deleteLater();
}
m_filterWidget = (widget);
auto old = ui->splitter->replaceWidget(0, m_filterWidget);
// because we replaced the widget we also need to delete it // because we replaced the widget we also need to delete it
if (old) { if (old) {
delete old; old->deleteLater();
} }
connect(ui->filterButton, &QPushButton::clicked, this, [this] { m_filterWidget->setHidden(!m_filterWidget->isHidden()); }); connect(ui->filterButton, &QPushButton::clicked, this, [this] { m_filterWidget->setHidden(!m_filterWidget->isHidden()); });
connect(m_filterWidget.get(), &ModFilterWidget::filterChanged, this, &FlamePage::triggerSearch); connect(m_filterWidget, &ModFilterWidget::filterChanged, this, &FlamePage::triggerSearch);
auto response = std::make_shared<QByteArray>(); auto response = std::make_shared<QByteArray>();
m_categoriesTask = FlameAPI::getCategories(response, ModPlatform::ResourceType::MODPACK); m_categoriesTask = FlameAPI::getCategories(response, ModPlatform::ResourceType::MODPACK);
QObject::connect(m_categoriesTask.get(), &Task::succeeded, [this, response]() { QObject::connect(m_categoriesTask.get(), &Task::succeeded, [this, response]() {

2
launcher/ui/pages/modplatform/flame/FlamePage.h
View File

@ -100,6 +100,6 @@ class FlamePage : public QWidget, public ModpackProviderBasePage {
// Used to do instant searching with a delay to cache quick changes // Used to do instant searching with a delay to cache quick changes
QTimer m_search_timer; QTimer m_search_timer;
unique_qobject_ptr<ModFilterWidget> m_filterWidget; ModFilterWidget* m_filterWidget;
Task::Ptr m_categoriesTask; Task::Ptr m_categoriesTask;
}; };

4
launcher/ui/pages/modplatform/flame/FlameResourcePages.cpp
View File

@ -207,9 +207,9 @@ auto FlameShaderPackPage::shouldDisplay() const -> bool
return true; return true;
} }
unique_qobject_ptr<ModFilterWidget> FlameModPage::createFilterWidget() ModFilterWidget* FlameModPage::createFilterWidget()
{ {
return ModFilterWidget::create(&static_cast<MinecraftInstance&>(m_baseInstance), false, this); return new ModFilterWidget(&static_cast<MinecraftInstance&>(m_baseInstance), false, this);
} }
void FlameModPage::prepareProviderCategories() void FlameModPage::prepareProviderCategories()

2
launcher/ui/pages/modplatform/flame/FlameResourcePages.h
View File

@ -96,7 +96,7 @@ class FlameModPage : public ModPage {
[[nodiscard]] inline auto helpPage() const -> QString override { return "Mod-platform"; } [[nodiscard]] inline auto helpPage() const -> QString override { return "Mod-platform"; }
void openUrl(const QUrl& url) override; void openUrl(const QUrl& url) override;
unique_qobject_ptr<ModFilterWidget> createFilterWidget() override; ModFilterWidget* createFilterWidget() override;
protected: protected:
virtual void prepareProviderCategories() override; virtual void prepareProviderCategories() override;

12
launcher/ui/pages/modplatform/modrinth/ModrinthPage.cpp
View File

@ -391,17 +391,19 @@ QString ModrinthPage::getSerachTerm() const
void ModrinthPage::createFilterWidget() void ModrinthPage::createFilterWidget()
{ {
auto widget = ModFilterWidget::create(nullptr, true, this); auto widget = new ModFilterWidget(nullptr, true, this);
m_filterWidget.swap(widget); if (m_filterWidget)
auto old = ui->splitter->replaceWidget(0, m_filterWidget.get()); m_filterWidget->deleteLater();
m_filterWidget = widget;
auto old = ui->splitter->replaceWidget(0, m_filterWidget);
// because we replaced the widget we also need to delete it // because we replaced the widget we also need to delete it
if (old) { if (old) {
delete old; old->deleteLater();
} }
connect(ui->filterButton, &QPushButton::clicked, this, [this] { m_filterWidget->setHidden(!m_filterWidget->isHidden()); }); connect(ui->filterButton, &QPushButton::clicked, this, [this] { m_filterWidget->setHidden(!m_filterWidget->isHidden()); });
connect(m_filterWidget.get(), &ModFilterWidget::filterChanged, this, &ModrinthPage::triggerSearch); connect(m_filterWidget, &ModFilterWidget::filterChanged, this, &ModrinthPage::triggerSearch);
auto response = std::make_shared<QByteArray>(); auto response = std::make_shared<QByteArray>();
m_categoriesTask = ModrinthAPI::getModCategories(response); m_categoriesTask = ModrinthAPI::getModCategories(response);
QObject::connect(m_categoriesTask.get(), &Task::succeeded, [this, response]() { QObject::connect(m_categoriesTask.get(), &Task::succeeded, [this, response]() {

2
launcher/ui/pages/modplatform/modrinth/ModrinthPage.h
View File

@ -103,6 +103,6 @@ class ModrinthPage : public QWidget, public ModpackProviderBasePage {
// Used to do instant searching with a delay to cache quick changes // Used to do instant searching with a delay to cache quick changes
QTimer m_search_timer; QTimer m_search_timer;
unique_qobject_ptr<ModFilterWidget> m_filterWidget; ModFilterWidget* m_filterWidget;
Task::Ptr m_categoriesTask; Task::Ptr m_categoriesTask;
}; };

4
launcher/ui/pages/modplatform/modrinth/ModrinthResourcePages.cpp
View File

@ -142,9 +142,9 @@ auto ModrinthShaderPackPage::shouldDisplay() const -> bool
return true; return true;
} }
unique_qobject_ptr<ModFilterWidget> ModrinthModPage::createFilterWidget() ModFilterWidget* ModrinthModPage::createFilterWidget()
{ {
return ModFilterWidget::create(&static_cast<MinecraftInstance&>(m_baseInstance), true, this); return new ModFilterWidget(&static_cast<MinecraftInstance&>(m_baseInstance), true, this);
} }
void ModrinthModPage::prepareProviderCategories() void ModrinthModPage::prepareProviderCategories()

2
launcher/ui/pages/modplatform/modrinth/ModrinthResourcePages.h
View File

@ -94,7 +94,7 @@ class ModrinthModPage : public ModPage {
[[nodiscard]] inline auto helpPage() const -> QString override { return "Mod-platform"; } [[nodiscard]] inline auto helpPage() const -> QString override { return "Mod-platform"; }
unique_qobject_ptr<ModFilterWidget> createFilterWidget() override; ModFilterWidget* createFilterWidget() override;
protected: protected:
virtual void prepareProviderCategories() override; virtual void prepareProviderCategories() override;

5
launcher/ui/widgets/ModFilterWidget.cpp
View File

@ -49,11 +49,6 @@
#include "Application.h" #include "Application.h"
#include "minecraft/PackProfile.h" #include "minecraft/PackProfile.h"
unique_qobject_ptr<ModFilterWidget> ModFilterWidget::create(MinecraftInstance* instance, bool extended, QWidget* parent)
{
return unique_qobject_ptr<ModFilterWidget>(new ModFilterWidget(instance, extended, parent));
}
class VersionBasicModel : public QIdentityProxyModel { class VersionBasicModel : public QIdentityProxyModel {
Q_OBJECT Q_OBJECT

4
launcher/ui/widgets/ModFilterWidget.h
View File

@ -83,7 +83,7 @@ class ModFilterWidget : public QTabWidget {
} }
}; };
static unique_qobject_ptr<ModFilterWidget> create(MinecraftInstance* instance, bool extended, QWidget* parent = nullptr); ModFilterWidget(MinecraftInstance* instance, bool extendedSupport, QWidget* parent = nullptr);
virtual ~ModFilterWidget(); virtual ~ModFilterWidget();
auto getFilter() -> std::shared_ptr<Filter>; auto getFilter() -> std::shared_ptr<Filter>;
@ -96,8 +96,6 @@ class ModFilterWidget : public QTabWidget {
void setCategories(const QList<ModPlatform::Category>&); void setCategories(const QList<ModPlatform::Category>&);
private: private:
ModFilterWidget(MinecraftInstance* instance, bool extendedSupport, QWidget* parent = nullptr);
void loadVersionList(); void loadVersionList();
void prepareBasicFilter(); void prepareBasicFilter();