diff options
| author | Thomas Hartmann <Thomas.Hartmann@nokia.com> | 2012-02-14 15:21:21 +0100 |
|---|---|---|
| committer | Thomas Hartmann <Thomas.Hartmann@nokia.com> | 2012-02-15 10:56:54 +0100 |
| commit | 0f073e4a35963d05727871a31898eaf6daf59dc8 (patch) | |
| tree | 173179bf4cea1155759e007085092eb32f3e8251 /src/plugins/qt4projectmanager/wizards/testwizard.cpp | |
| parent | c4b631f97575cca5921a0c8645428354dbcf6745 (diff) | |
| download | qt-creator-0f073e4a35963d05727871a31898eaf6daf59dc8.tar.gz | |
Wizards: avoiding code duplication
I moved setPath(wizardDialogParameters.defaultPath()) into
the constructor of BaseProjectWizardDialog.
I created addExtensionPages() instead of having of having the
foreach everywhere. Moving the call into the constructor of
BaseProjectWizardDialog is not trivial since a lot of derived classes
rely on execution order and the order is often different.
Entangling this is not trivial and easily might break functionality.
Change-Id: I48dddaf72caea84da783dc9e2f42f2c7eff1c0ce
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@nokia.com>
Diffstat (limited to 'src/plugins/qt4projectmanager/wizards/testwizard.cpp')
| -rw-r--r-- | src/plugins/qt4projectmanager/wizards/testwizard.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/plugins/qt4projectmanager/wizards/testwizard.cpp b/src/plugins/qt4projectmanager/wizards/testwizard.cpp index 93c45f7bf6..012fcb9ff3 100644 --- a/src/plugins/qt4projectmanager/wizards/testwizard.cpp +++ b/src/plugins/qt4projectmanager/wizards/testwizard.cpp @@ -63,7 +63,6 @@ QWizard *TestWizard::createWizardDialog(QWidget *parent, const Core::WizardDialogParameters &wizardDialogParameters) const { TestWizardDialog *dialog = new TestWizardDialog(displayName(), icon(), parent, wizardDialogParameters); - dialog->setPath(wizardDialogParameters.defaultPath()); dialog->setProjectName(TestWizardDialog::uniqueProjectName(wizardDialogParameters.defaultPath())); return dialog; } |