diff options
author | hjk <hjk@qt.io> | 2019-02-25 18:35:34 +0100 |
---|---|---|
committer | hjk <hjk@qt.io> | 2019-02-26 10:41:50 +0000 |
commit | 4d2981b1f4c9620584a5d2d1063715dfaee31fe5 (patch) | |
tree | 2bddc182d0ed17f98a472da9edb3db8d211f9f3a /src | |
parent | 65be7c2915969d9491bf75216c1a2b355be72207 (diff) | |
download | qt-creator-4d2981b1f4c9620584a5d2d1063715dfaee31fe5.tar.gz |
ProjectExplorer: Un-protect ProjectNode constructor
Not much protection needed, and not provided either, as seen in
the removed workarounds.
Change-Id: Ib33f6481cdb04481c0324bdeb57eb50143ecb62f
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
Diffstat (limited to 'src')
4 files changed, 6 insertions, 25 deletions
diff --git a/src/plugins/compilationdatabaseprojectmanager/compilationdatabaseproject.cpp b/src/plugins/compilationdatabaseprojectmanager/compilationdatabaseproject.cpp index 34e5922b6a..d54b51cc4b 100644 --- a/src/plugins/compilationdatabaseprojectmanager/compilationdatabaseproject.cpp +++ b/src/plugins/compilationdatabaseprojectmanager/compilationdatabaseproject.cpp @@ -61,13 +61,6 @@ namespace CompilationDatabaseProjectManager { namespace Internal { namespace { -class DBProjectNode : public ProjectNode -{ -public: - explicit DBProjectNode(const Utils::FileName &projectFilePath) - : ProjectNode(projectFilePath) - {} -}; QStringList jsonObjectFlags(const QJsonObject &object) { @@ -356,7 +349,7 @@ void CompilationDatabaseProject::buildTreeAndProjectParts(const Utils::FileName return; } - auto root = std::make_unique<DBProjectNode>(projectDirectory()); + auto root = std::make_unique<ProjectNode>(projectDirectory()); CppTools::KitInfo kitInfo(this); QTC_ASSERT(kitInfo.isValid(), return); diff --git a/src/plugins/projectexplorer/project.cpp b/src/plugins/projectexplorer/project.cpp index a030d9b2f3..4c3f867c51 100644 --- a/src/plugins/projectexplorer/project.cpp +++ b/src/plugins/projectexplorer/project.cpp @@ -958,12 +958,6 @@ public: bool needsConfiguration() const final { return false; } }; -class TestProjectNode : public ProjectNode -{ -public: - TestProjectNode(const Utils::FileName &dir) : ProjectNode(dir) { } -}; - void ProjectExplorerPlugin::testProject_setup() { TestProject project; @@ -1060,7 +1054,7 @@ void ProjectExplorerPlugin::testProject_parsingFail() std::unique_ptr<ProjectNode> createFileTree(Project *project) { - std::unique_ptr<ProjectNode> root = std::make_unique<TestProjectNode>(project->projectDirectory()); + std::unique_ptr<ProjectNode> root = std::make_unique<ProjectNode>(project->projectDirectory()); std::vector<std::unique_ptr<FileNode>> nodes; nodes.emplace_back(std::make_unique<FileNode>(TEST_PROJECT_PATH, FileType::Project)); nodes.emplace_back(std::make_unique<FileNode>(TEST_PROJECT_CPP_FILE, FileType::Source)); @@ -1080,7 +1074,7 @@ void ProjectExplorerPlugin::testProject_projectTree() QCOMPARE(fileSpy.count(), 0); QVERIFY(!project.rootProjectNode()); - project.setRootProjectNode(std::make_unique<TestProjectNode>(project.projectDirectory())); + project.setRootProjectNode(std::make_unique<ProjectNode>(project.projectDirectory())); QCOMPARE(fileSpy.count(), 0); QVERIFY(!project.rootProjectNode()); diff --git a/src/plugins/projectexplorer/projectnodes.h b/src/plugins/projectexplorer/projectnodes.h index 6c59686115..d1a25134ae 100644 --- a/src/plugins/projectexplorer/projectnodes.h +++ b/src/plugins/projectexplorer/projectnodes.h @@ -322,6 +322,8 @@ private: class PROJECTEXPLORER_EXPORT ProjectNode : public FolderNode { public: + explicit ProjectNode(const Utils::FileName &projectFilePath); + virtual bool canAddSubProject(const QString &proFilePath) const; virtual bool addSubProject(const QString &proFile); virtual QStringList subProjectFileNamePatterns() const; @@ -353,7 +355,6 @@ public: virtual bool setData(Core::Id role, const QVariant &value) const; protected: - explicit ProjectNode(const Utils::FileName &projectFilePath); QString m_target; }; diff --git a/src/plugins/qmlprofiler/tests/qmlprofilerdetailsrewriter_test.cpp b/src/plugins/qmlprofiler/tests/qmlprofilerdetailsrewriter_test.cpp index 7ba3b02cdb..0f07737f5c 100644 --- a/src/plugins/qmlprofiler/tests/qmlprofilerdetailsrewriter_test.cpp +++ b/src/plugins/qmlprofiler/tests/qmlprofilerdetailsrewriter_test.cpp @@ -41,13 +41,6 @@ namespace QmlProfiler { namespace Internal { -class DummyProjectNode : public ProjectExplorer::ProjectNode -{ -public: - DummyProjectNode(const Utils::FileName &file) : ProjectExplorer::ProjectNode(file) - {} -}; - class DummyProject : public ProjectExplorer::Project { Q_OBJECT @@ -57,7 +50,7 @@ public: { auto fileNode = std::make_unique<ProjectExplorer::FileNode>(file, ProjectExplorer::FileType::Source); - auto root = std::make_unique<DummyProjectNode>(file); + auto root = std::make_unique<ProjectExplorer::ProjectNode>(file); root->addNode(std::move(fileNode)); fileNode = std::make_unique<ProjectExplorer::FileNode>( Utils::FileName::fromLatin1( |