feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

CMake: Remove CMakeProject::buildTargetTitles

Browse Source 
The only user can also get this information from the relevant
CMakeBuildConfiguration.

Change-Id: I80e176ef0a8bc427f6adbf75b20e29f38d7b949f
Reviewed-by: Eike Ziller <eike.ziller@qt.io>
This commit is contained in:
Tobias Hunger
2019-08-12 14:12:01 +02:00
parent dbfa55f5fc
commit a90b1c26b1
4 changed files with 10 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/plugins/cmakeprojectmanager/cmakebuildstep.cpp
View File

@@ -108,8 +108,7 @@ void CMakeBuildStep::handleBuildTargetChanges(bool success)
{
if (!success)
return; // Do not change when parsing failed.
if (!isCurrentExecutableTarget(m_buildTarget)
&& !static_cast<CMakeProject *>(project())->buildTargetTitles().contains(m_buildTarget)) {
if (!isCurrentExecutableTarget(m_buildTarget) && !knownBuildTargets().contains(m_buildTarget)) {
setBuildTarget(defaultBuildTarget());
}
emit buildTargetsChanged();
@@ -374,6 +373,12 @@ Utils::CommandLine CMakeBuildStep::cmakeCommand(RunConfiguration *rc) const
return cmd;
}
QStringList CMakeBuildStep::knownBuildTargets()
{
auto bc = qobject_cast<CMakeBuildConfiguration *>(buildConfiguration());
return bc ? bc->buildTargetTitles() : QStringList();
}
QString CMakeBuildStep::cleanTarget()
{
return QString("clean");
@@ -472,8 +477,7 @@ void CMakeBuildStepConfigWidget::buildTargetsChanged()
QSignalBlocker blocker(m_buildTargetsList);
m_buildTargetsList->clear();
auto pro = static_cast<CMakeProject *>(m_buildStep->project());
QStringList targetList = pro->buildTargetTitles();
QStringList targetList = m_buildStep->knownBuildTargets();
targetList.sort();
QFont italics;

2
src/plugins/cmakeprojectmanager/cmakebuildstep.h
View File

@@ -71,6 +71,8 @@ public:
Utils::CommandLine cmakeCommand(ProjectExplorer::RunConfiguration *rc) const;
QStringList knownBuildTargets();
QVariantMap toMap() const override;
static QString cleanTarget();

6
src/plugins/cmakeprojectmanager/cmakeproject.cpp
View File

@@ -392,12 +392,6 @@ void CMakeProject::startParsing(int reparseParameters)
bc->m_buildDirManager.parse(reparseParameters);
}
QStringList CMakeProject::buildTargetTitles() const
{
CMakeBuildConfiguration *bc = activeBc(this);
return bc ? bc->buildTargetTitles() : QStringList();
}
bool CMakeProject::setupTarget(Target *t)
{
t->updateDefaultBuildConfigurations();

2
src/plugins/cmakeprojectmanager/cmakeproject.h
View File

@@ -63,8 +63,6 @@ public:
explicit CMakeProject(const Utils::FilePath &filename);
~CMakeProject() final;
QStringList buildTargetTitles() const;
ProjectExplorer::Tasks projectIssues(const ProjectExplorer::Kit *k) const final;
void runCMake();