Replace BuildConfigWidge with NamedWidget

They have a identical interface.

Change-Id: Ia626496fbaffedefff6ee20b958cd505085d89f7
Reviewed-by: Tobias Hunger <tobias.hunger@digia.com>
This commit is contained in:
Daniel Teske
2012-11-08 18:02:50 +01:00
parent ea640a5aea
commit 51bca85117
22 changed files with 37 additions and 66 deletions

View File

@@ -65,7 +65,7 @@ AutotoolsBuildConfiguration::AutotoolsBuildConfiguration(ProjectExplorer::Target
m_buildDirectory = project->defaultBuildDirectory(); m_buildDirectory = project->defaultBuildDirectory();
} }
BuildConfigWidget *AutotoolsBuildConfiguration::createConfigWidget() NamedWidget *AutotoolsBuildConfiguration::createConfigWidget()
{ {
return new AutotoolsBuildSettingsWidget(this); return new AutotoolsBuildSettingsWidget(this);
} }

View File

@@ -48,7 +48,7 @@ class AutotoolsBuildConfiguration : public ProjectExplorer::BuildConfiguration
public: public:
explicit AutotoolsBuildConfiguration(ProjectExplorer::Target *parent); explicit AutotoolsBuildConfiguration(ProjectExplorer::Target *parent);
ProjectExplorer::BuildConfigWidget *createConfigWidget(); ProjectExplorer::NamedWidget *createConfigWidget();
QString buildDirectory() const; QString buildDirectory() const;
void setBuildDirectory(const QString &buildDirectory); void setBuildDirectory(const QString &buildDirectory);

View File

@@ -32,7 +32,7 @@
#ifndef AUTOTOOLSBUILDSETTINGSWIDGET_H #ifndef AUTOTOOLSBUILDSETTINGSWIDGET_H
#define AUTOTOOLSBUILDSETTINGSWIDGET_H #define AUTOTOOLSBUILDSETTINGSWIDGET_H
#include <projectexplorer/buildstep.h> #include <projectexplorer/namedwidget.h>
QT_BEGIN_NAMESPACE QT_BEGIN_NAMESPACE
class QComboBox; class QComboBox;
@@ -52,7 +52,7 @@ class AutotoolsBuildConfiguration;
* *
* Provides an editor to configure the build directory and build steps. * Provides an editor to configure the build directory and build steps.
*/ */
class AutotoolsBuildSettingsWidget : public ProjectExplorer::BuildConfigWidget class AutotoolsBuildSettingsWidget : public ProjectExplorer::NamedWidget
{ {
Q_OBJECT Q_OBJECT

View File

@@ -106,7 +106,7 @@ void CMakeBuildConfiguration::setUseNinja(bool useNninja)
CMakeBuildConfiguration::~CMakeBuildConfiguration() CMakeBuildConfiguration::~CMakeBuildConfiguration()
{ } { }
ProjectExplorer::BuildConfigWidget *CMakeBuildConfiguration::createConfigWidget() ProjectExplorer::NamedWidget *CMakeBuildConfiguration::createConfigWidget()
{ {
return new CMakeBuildSettingsWidget(this); return new CMakeBuildSettingsWidget(this);
} }

View File

@@ -51,7 +51,7 @@ public:
CMakeBuildConfiguration(ProjectExplorer::Target *parent); CMakeBuildConfiguration(ProjectExplorer::Target *parent);
~CMakeBuildConfiguration(); ~CMakeBuildConfiguration();
ProjectExplorer::BuildConfigWidget *createConfigWidget(); ProjectExplorer::NamedWidget *createConfigWidget();
QString buildDirectory() const; QString buildDirectory() const;
void setBuildDirectory(const QString &buildDirectory); void setBuildDirectory(const QString &buildDirectory);

View File

@@ -37,8 +37,8 @@
#include <projectexplorer/project.h> #include <projectexplorer/project.h>
#include <projectexplorer/projectnodes.h> #include <projectexplorer/projectnodes.h>
#include <projectexplorer/buildstep.h>
#include <projectexplorer/buildconfiguration.h> #include <projectexplorer/buildconfiguration.h>
#include <projectexplorer/namedwidget.h>
#include <coreplugin/idocument.h> #include <coreplugin/idocument.h>
#include <coreplugin/editormanager/editormanager.h> #include <coreplugin/editormanager/editormanager.h>
#include <coreplugin/editormanager/ieditor.h> #include <coreplugin/editormanager/ieditor.h>
@@ -224,7 +224,7 @@ private:
QString m_fileName; QString m_fileName;
}; };
class CMakeBuildSettingsWidget : public ProjectExplorer::BuildConfigWidget class CMakeBuildSettingsWidget : public ProjectExplorer::NamedWidget
{ {
Q_OBJECT Q_OBJECT
public: public:

View File

@@ -105,7 +105,7 @@ void GenericBuildConfiguration::setBuildDirectory(const QString &buildDirectory)
emit buildDirectoryChanged(); emit buildDirectoryChanged();
} }
BuildConfigWidget *GenericBuildConfiguration::createConfigWidget() NamedWidget *GenericBuildConfiguration::createConfigWidget()
{ {
return new GenericBuildSettingsWidget(this); return new GenericBuildSettingsWidget(this);
} }

View File

@@ -31,7 +31,7 @@
#define GENERICBUILDCONFIGURATION_H #define GENERICBUILDCONFIGURATION_H
#include <projectexplorer/buildconfiguration.h> #include <projectexplorer/buildconfiguration.h>
#include <projectexplorer/buildstep.h> // for BuildConfigWidget #include <projectexplorer/namedwidget.h>
namespace Utils { class PathChooser; } namespace Utils { class PathChooser; }
@@ -49,7 +49,7 @@ class GenericBuildConfiguration : public ProjectExplorer::BuildConfiguration
public: public:
explicit GenericBuildConfiguration(ProjectExplorer::Target *parent); explicit GenericBuildConfiguration(ProjectExplorer::Target *parent);
ProjectExplorer::BuildConfigWidget *createConfigWidget(); ProjectExplorer::NamedWidget *createConfigWidget();
QString buildDirectory() const; QString buildDirectory() const;
QString rawBuildDirectory() const; QString rawBuildDirectory() const;
@@ -89,7 +89,7 @@ private:
bool canHandle(const ProjectExplorer::Target *t) const; bool canHandle(const ProjectExplorer::Target *t) const;
}; };
class GenericBuildSettingsWidget : public ProjectExplorer::BuildConfigWidget class GenericBuildSettingsWidget : public ProjectExplorer::NamedWidget
{ {
Q_OBJECT Q_OBJECT

View File

@@ -117,9 +117,9 @@ BuildConfiguration::~BuildConfiguration()
delete m_macroExpander; delete m_macroExpander;
} }
QList<BuildConfigWidget*> BuildConfiguration::createSubConfigWidgets() QList<NamedWidget *> BuildConfiguration::createSubConfigWidgets()
{ {
return QList<BuildConfigWidget *>() << new ProjectExplorer::BuildEnvironmentWidget(this); return QList<NamedWidget *>() << new ProjectExplorer::BuildEnvironmentWidget(this);
} }
Utils::AbstractMacroExpander *BuildConfiguration::macroExpander() Utils::AbstractMacroExpander *BuildConfiguration::macroExpander()

View File

@@ -44,7 +44,7 @@ class AbstractMacroExpander;
namespace ProjectExplorer { namespace ProjectExplorer {
class BuildConfiguration; class BuildConfiguration;
class BuildConfigWidget; class NamedWidget;
class BuildStepList; class BuildStepList;
class Kit; class Kit;
class Target; class Target;
@@ -60,8 +60,8 @@ public:
virtual QString buildDirectory() const = 0; virtual QString buildDirectory() const = 0;
virtual BuildConfigWidget *createConfigWidget() = 0; virtual ProjectExplorer::NamedWidget *createConfigWidget() = 0;
virtual QList<BuildConfigWidget*> createSubConfigWidgets(); virtual QList<NamedWidget *> createSubConfigWidgets();
// Maybe the BuildConfiguration is not the best place for the environment // Maybe the BuildConfiguration is not the best place for the environment
Utils::Environment baseEnvironment() const; Utils::Environment baseEnvironment() const;

View File

@@ -31,6 +31,7 @@
#define BUILDENVIRONMENTWIDGET_H #define BUILDENVIRONMENTWIDGET_H
#include <projectexplorer/buildstep.h> #include <projectexplorer/buildstep.h>
#include <projectexplorer/namedwidget.h>
QT_BEGIN_NAMESPACE QT_BEGIN_NAMESPACE
class QCheckBox; class QCheckBox;
@@ -41,7 +42,7 @@ namespace ProjectExplorer {
class EnvironmentWidget; class EnvironmentWidget;
class BuildConfiguration; class BuildConfiguration;
class PROJECTEXPLORER_EXPORT BuildEnvironmentWidget : public BuildConfigWidget class PROJECTEXPLORER_EXPORT BuildEnvironmentWidget : public NamedWidget
{ {
Q_OBJECT Q_OBJECT

View File

@@ -175,7 +175,7 @@ BuildSettingsWidget::BuildSettingsWidget(Target *target) :
connect(m_target, SIGNAL(kitChanged()), this, SLOT(updateAddButtonMenu())); connect(m_target, SIGNAL(kitChanged()), this, SLOT(updateAddButtonMenu()));
} }
void BuildSettingsWidget::addSubWidget(BuildConfigWidget *widget) void BuildSettingsWidget::addSubWidget(NamedWidget *widget)
{ {
widget->setContentsMargins(0, 10, 0, 0); widget->setContentsMargins(0, 10, 0, 0);
@@ -205,7 +205,7 @@ void BuildSettingsWidget::clear()
m_labels.clear(); m_labels.clear();
} }
QList<BuildConfigWidget *> BuildSettingsWidget::subWidgets() const QList<NamedWidget *> BuildSettingsWidget::subWidgets() const
{ {
return m_subWidgets; return m_subWidgets;
} }
@@ -239,14 +239,14 @@ void BuildSettingsWidget::updateBuildSettings()
return; return;
// Add pages // Add pages
BuildConfigWidget *generalConfigWidget = m_buildConfiguration->createConfigWidget(); NamedWidget *generalConfigWidget = m_buildConfiguration->createConfigWidget();
addSubWidget(generalConfigWidget); addSubWidget(generalConfigWidget);
addSubWidget(new BuildStepsPage(m_buildConfiguration, Core::Id(Constants::BUILDSTEPS_BUILD))); addSubWidget(new BuildStepsPage(m_buildConfiguration, Core::Id(Constants::BUILDSTEPS_BUILD)));
addSubWidget(new BuildStepsPage(m_buildConfiguration, Core::Id(Constants::BUILDSTEPS_CLEAN))); addSubWidget(new BuildStepsPage(m_buildConfiguration, Core::Id(Constants::BUILDSTEPS_CLEAN)));
QList<BuildConfigWidget *> subConfigWidgets = m_buildConfiguration->createSubConfigWidgets(); QList<NamedWidget *> subConfigWidgets = m_buildConfiguration->createSubConfigWidgets();
foreach (BuildConfigWidget *subConfigWidget, subConfigWidgets) foreach (NamedWidget *subConfigWidget, subConfigWidgets)
addSubWidget(subConfigWidget); addSubWidget(subConfigWidget);
} }

View File

@@ -45,8 +45,8 @@ QT_END_NAMESPACE
namespace ProjectExplorer { namespace ProjectExplorer {
class BuildConfiguration; class BuildConfiguration;
class BuildConfigWidget;
class IBuildStepFactory; class IBuildStepFactory;
class NamedWidget;
namespace Internal { namespace Internal {
@@ -73,8 +73,8 @@ public:
~BuildSettingsWidget(); ~BuildSettingsWidget();
void clear(); void clear();
void addSubWidget(BuildConfigWidget *widget); void addSubWidget(ProjectExplorer::NamedWidget *widget);
QList<BuildConfigWidget *> subWidgets() const; QList<ProjectExplorer::NamedWidget *> subWidgets() const;
private slots: private slots:
void updateBuildSettings(); void updateBuildSettings();
@@ -103,7 +103,7 @@ private:
QComboBox *m_buildConfigurationComboBox; QComboBox *m_buildConfigurationComboBox;
QMenu *m_addButtonMenu; QMenu *m_addButtonMenu;
QList<BuildConfigWidget *> m_subWidgets; QList<NamedWidget *> m_subWidgets;
QList<QLabel *> m_labels; QList<QLabel *> m_labels;
}; };

View File

@@ -118,36 +118,6 @@ public:
virtual BuildStep *clone(BuildStepList *parent, BuildStep *product) = 0; virtual BuildStep *clone(BuildStepList *parent, BuildStep *product) = 0;
}; };
class PROJECTEXPLORER_EXPORT BuildConfigWidget
: public QWidget
{
Q_OBJECT
public:
BuildConfigWidget()
:QWidget(0)
{}
QString displayName() const
{
return m_displayName;
}
signals:
void displayNameChanged(const QString &);
protected:
void setDisplayName(const QString &displayName)
{
if (m_displayName == displayName)
return;
m_displayName = displayName;
emit displayNameChanged(m_displayName);
}
private:
QString m_displayName;
};
class PROJECTEXPLORER_EXPORT BuildStepConfigWidget class PROJECTEXPLORER_EXPORT BuildStepConfigWidget
: public QWidget : public QWidget
{ {

View File

@@ -492,7 +492,7 @@ void BuildStepListWidget::updateBuildStepButtonsState()
} }
BuildStepsPage::BuildStepsPage(BuildConfiguration *bc, Core::Id id) : BuildStepsPage::BuildStepsPage(BuildConfiguration *bc, Core::Id id) :
BuildConfigWidget(), NamedWidget(),
m_id(id), m_id(id),
m_widget(new BuildStepListWidget(this)) m_widget(new BuildStepListWidget(this))
{ {

View File

@@ -151,7 +151,7 @@ namespace Ui {
class BuildStepsPage; class BuildStepsPage;
} }
class BuildStepsPage : public BuildConfigWidget class BuildStepsPage : public NamedWidget
{ {
Q_OBJECT Q_OBJECT

View File

@@ -45,11 +45,11 @@ class Context;
namespace ProjectExplorer { namespace ProjectExplorer {
class BuildConfigWidget;
class IProjectManager; class IProjectManager;
class EditorConfiguration; class EditorConfiguration;
class ProjectNode; class ProjectNode;
class Kit; class Kit;
class NamedWidget;
class Target; class Target;
class ProjectPrivate; class ProjectPrivate;

View File

@@ -49,7 +49,7 @@ class DeployConfigurationFactory;
class IRunConfigurationFactory; class IRunConfigurationFactory;
class Kit; class Kit;
class Project; class Project;
class BuildConfigWidget; class NamedWidget;
class TargetPrivate; class TargetPrivate;

View File

@@ -184,7 +184,7 @@ void Qt4BuildConfiguration::emitBuildDirectoryChanged()
} }
} }
BuildConfigWidget *Qt4BuildConfiguration::createConfigWidget() NamedWidget *Qt4BuildConfiguration::createConfigWidget()
{ {
return new Qt4ProjectConfigWidget(this); return new Qt4ProjectConfigWidget(this);
} }

View File

@@ -55,7 +55,7 @@ public:
explicit Qt4BuildConfiguration(ProjectExplorer::Target *target); explicit Qt4BuildConfiguration(ProjectExplorer::Target *target);
~Qt4BuildConfiguration(); ~Qt4BuildConfiguration();
ProjectExplorer::BuildConfigWidget *createConfigWidget(); ProjectExplorer::NamedWidget *createConfigWidget();
QString buildDirectory() const; QString buildDirectory() const;
bool shadowBuild() const; bool shadowBuild() const;
QString shadowBuildDirectory() const; QString shadowBuildDirectory() const;

View File

@@ -63,7 +63,7 @@ using namespace Qt4ProjectManager::Internal;
using namespace ProjectExplorer; using namespace ProjectExplorer;
Qt4ProjectConfigWidget::Qt4ProjectConfigWidget(Qt4BuildConfiguration *bc) Qt4ProjectConfigWidget::Qt4ProjectConfigWidget(Qt4BuildConfiguration *bc)
: BuildConfigWidget(), : NamedWidget(),
m_buildConfiguration(bc), m_buildConfiguration(bc),
m_ignoreChange(false) m_ignoreChange(false)
{ {

View File

@@ -30,7 +30,7 @@
#ifndef QT4PROJECTCONFIGWIDGET_H #ifndef QT4PROJECTCONFIGWIDGET_H
#define QT4PROJECTCONFIGWIDGET_H #define QT4PROJECTCONFIGWIDGET_H
#include <projectexplorer/buildstep.h> #include <projectexplorer/namedwidget.h>
QT_BEGIN_NAMESPACE QT_BEGIN_NAMESPACE
class QAbstractButton; class QAbstractButton;
@@ -49,7 +49,7 @@ namespace Ui {
class Qt4ProjectConfigWidget; class Qt4ProjectConfigWidget;
} }
class Qt4ProjectConfigWidget : public ProjectExplorer::BuildConfigWidget class Qt4ProjectConfigWidget : public ProjectExplorer::NamedWidget
{ {
Q_OBJECT Q_OBJECT
public: public: