From d1d584d16b252218de5f7e13cc9d47e71f465043 Mon Sep 17 00:00:00 2001 From: Lioncash Date: Sun, 13 May 2018 15:55:51 -0400 Subject: [PATCH] FilesystemWidget: Replace DiscIO includes with forward declarations Avoids the need to use an ugly ifdef around a header, and also resolves an indirect inclusion in PropertiesDialog. --- Source/Core/DolphinQt2/Config/FilesystemWidget.cpp | 5 ++++- Source/Core/DolphinQt2/Config/FilesystemWidget.h | 11 ++++------- Source/Core/DolphinQt2/Config/PropertiesDialog.cpp | 2 ++ 3 files changed, 10 insertions(+), 8 deletions(-) diff --git a/Source/Core/DolphinQt2/Config/FilesystemWidget.cpp b/Source/Core/DolphinQt2/Config/FilesystemWidget.cpp index dc27b9df912a..83d473ff3337 100644 --- a/Source/Core/DolphinQt2/Config/FilesystemWidget.cpp +++ b/Source/Core/DolphinQt2/Config/FilesystemWidget.cpp @@ -19,8 +19,9 @@ #include #include "DiscIO/DiscExtractor.h" -#include "DiscIO/Enums.h" #include "DiscIO/Filesystem.h" +#include "DiscIO/Volume.h" + #include "DolphinQt2/QtUtils/ActionHelper.h" #include "DolphinQt2/Resources.h" @@ -45,6 +46,8 @@ FilesystemWidget::FilesystemWidget(const UICommon::GameFile& game) PopulateView(); } +FilesystemWidget::~FilesystemWidget() = default; + void FilesystemWidget::CreateWidgets() { auto* layout = new QVBoxLayout; diff --git a/Source/Core/DolphinQt2/Config/FilesystemWidget.h b/Source/Core/DolphinQt2/Config/FilesystemWidget.h index a9d7a998f0a6..ba1a16db367c 100644 --- a/Source/Core/DolphinQt2/Config/FilesystemWidget.h +++ b/Source/Core/DolphinQt2/Config/FilesystemWidget.h @@ -7,12 +7,6 @@ #include #include -// Qt versions prior to 5.9 don't support C++17 nested namespaces in moc so we have to if-guard -// this header -#ifndef Q_MOC_RUN -#include "DiscIO/Volume.h" -#endif - #include "UICommon/GameFile.h" class QStandardItem; @@ -22,14 +16,17 @@ class QTreeView; namespace DiscIO { class FileInfo; +class Volume; + struct Partition; -}; // namespace DiscIO +} // namespace DiscIO class FilesystemWidget final : public QWidget { Q_OBJECT public: explicit FilesystemWidget(const UICommon::GameFile& game); + ~FilesystemWidget() override; private: void CreateWidgets(); diff --git a/Source/Core/DolphinQt2/Config/PropertiesDialog.cpp b/Source/Core/DolphinQt2/Config/PropertiesDialog.cpp index e80b6259e49b..a5c31d34d851 100644 --- a/Source/Core/DolphinQt2/Config/PropertiesDialog.cpp +++ b/Source/Core/DolphinQt2/Config/PropertiesDialog.cpp @@ -7,6 +7,8 @@ #include #include +#include "DiscIO/Enums.h" + #include "DolphinQt2/Config/ARCodeWidget.h" #include "DolphinQt2/Config/FilesystemWidget.h" #include "DolphinQt2/Config/GameConfigWidget.h"