From 114363677505011b3dd5a8cb5b64ea927f3cebf2 Mon Sep 17 00:00:00 2001 From: Friedemann Kleint <Friedemann.Kleint@nokia.com> Date: Tue, 27 Jan 2009 09:18:14 +0100 Subject: [PATCH] Fixes: Reduce dependencies to Designer headers/Kick out the unused "Generate Code" action --- src/plugins/designer/formeditorw.cpp | 50 +---------------------- src/plugins/designer/formeditorw.h | 2 - src/plugins/designer/formwindoweditor.cpp | 27 ------------ src/plugins/designer/formwindoweditor.h | 2 - 4 files changed, 1 insertion(+), 80 deletions(-) diff --git a/src/plugins/designer/formeditorw.cpp b/src/plugins/designer/formeditorw.cpp index 58de33f2753..54a14350313 100644 --- a/src/plugins/designer/formeditorw.cpp +++ b/src/plugins/designer/formeditorw.cpp @@ -84,8 +84,6 @@ #include <QtCore/QDebug> #include <QtCore/QSettings> -enum { wantCodeGenerationAction = 0 }; - static const char *editorWidgetStateKeyC = "editorWidgetState"; static const char *settingsGroup = "Designer"; @@ -166,8 +164,7 @@ FormEditorW::FormEditorW() : m_core(Core::ICore::instance()), m_initStage(RegisterPlugins), m_actionGroupEditMode(0), - m_actionPrint(0), - m_actionGenerateCode(0) + m_actionPrint(0) { if (Designer::Constants::Internal::debug) qDebug() << Q_FUNC_INFO; @@ -420,13 +417,6 @@ void FormEditorW::setupActions() m_actionGroupPreviewInStyle = m_fwm->actionGroupPreviewInStyle(); mformtools->addMenu(createPreviewStyleMenu(am, m_actionGroupPreviewInStyle)); - // Disabled since we cannot reliably locate uic. - if (wantCodeGenerationAction) { - m_actionGenerateCode = new QAction(tr("View &code"), this); - addToolAction(m_actionGenerateCode, am, m_context, - QLatin1String("FormEditor.ViewCode"), mformtools); - connect(m_actionGenerateCode, SIGNAL(triggered()), this, SLOT(generateCode())); - } // Form settings createSeparator(this, am, m_context, medit, QLatin1String("FormEditor.Edit.Separator2"), Core::Constants::G_EDIT_OTHER); @@ -513,42 +503,6 @@ void FormEditorW::restoreSettings(const QSettings *s) } } -void FormEditorW::generateCode() -{ - const FormWindowEditor *fww = activeFormWindow(); - if (!fww) - return; - - bool ok = false; - QString errorMessage; - - do { - QByteArray header; - if (!fww->generateCode(header, errorMessage)) - break; - - QString tempPattern = QDir::tempPath(); - if (!tempPattern.endsWith(QDir::separator())) // platform-dependant - tempPattern += QDir::separator(); - tempPattern += QLatin1String("ui_headerXXXXXX.h"); - QTemporaryFile headerFile(tempPattern); - headerFile.setAutoRemove (false); - if (!headerFile.open() || !headerFile.write(header)) { - errorMessage = tr("Unable to write to a temporary file."); - break; - } - const QString headerFileName = headerFile.fileName(); - headerFile.close(); - Core::IEditor *eif = m_core->editorManager()->openEditor(headerFileName); - if (!eif) { - errorMessage = tr("Unable open %1.").arg(headerFileName); - break; - } - ok = true; - } while (false); - if (!ok) - critical(errorMessage); -} void FormEditorW::critical(const QString &errorMessage) { @@ -621,8 +575,6 @@ void FormEditorW::activeFormWindowChanged(QDesignerFormWindowInterface *afw) } m_actionPreview->setEnabled(foundFormWindow); - if (m_actionGenerateCode) - m_actionGenerateCode->setEnabled(foundFormWindow); m_actionGroupPreviewInStyle->setEnabled(foundFormWindow); } diff --git a/src/plugins/designer/formeditorw.h b/src/plugins/designer/formeditorw.h index f41d3554e25..89da92443ce 100644 --- a/src/plugins/designer/formeditorw.h +++ b/src/plugins/designer/formeditorw.h @@ -120,7 +120,6 @@ public: FormWindowEditor *activeFormWindow(); private slots: - void generateCode(); void activateEditMode(int id); void activateEditMode(QAction*); void activeFormWindowChanged(QDesignerFormWindowInterface *); @@ -160,7 +159,6 @@ private: QList<SettingsPage *> m_settingsPages; QActionGroup *m_actionGroupEditMode; QAction *m_actionPrint; - QAction *m_actionGenerateCode; QAction *m_actionPreview; QActionGroup *m_actionGroupPreviewInStyle; diff --git a/src/plugins/designer/formwindoweditor.cpp b/src/plugins/designer/formwindoweditor.cpp index be6b763e61c..22a848321b7 100644 --- a/src/plugins/designer/formwindoweditor.cpp +++ b/src/plugins/designer/formwindoweditor.cpp @@ -300,33 +300,6 @@ QWidget *FormWindowEditor::widget() return m_editorWidget; } -bool FormWindowEditor::generateCode(QByteArray &header, QString &errorMessage) const -{ - if (Designer::Constants::Internal::debug) - qDebug() << Q_FUNC_INFO; - - QString tempPattern = QDir::tempPath(); - if (!tempPattern.endsWith(QDir::separator())) // platform-dependant - tempPattern += QDir::separator(); - tempPattern += QLatin1String("formXXXXXX.ui"); - QTemporaryFile uiFile(tempPattern); - uiFile.setAutoRemove(true); - if (!uiFile.open()) { - errorMessage = tr("Unable to write to a temporary file."); - return false; - } - if (!m_file->writeFile(uiFile, errorMessage)) { - errorMessage = tr("Unable to write to a temporary file."); - return false; - } - const QString uiFileName = uiFile.fileName(); - uiFile.close(); - - if (!qdesigner_internal::runUIC(uiFileName, qdesigner_internal::UIC_GenerateCode, header, errorMessage)) - return false; - - return true; -} QDesignerFormWindowInterface *FormWindowEditor::formWindow() const { diff --git a/src/plugins/designer/formwindoweditor.h b/src/plugins/designer/formwindoweditor.h index 6a449aeb96f..e854ccac915 100644 --- a/src/plugins/designer/formwindoweditor.h +++ b/src/plugins/designer/formwindoweditor.h @@ -87,8 +87,6 @@ public: QList<int> context() const; QWidget *widget(); - // Internal - bool generateCode(QByteArray &header, QString &errorMessage) const; QDesignerFormWindowInterface *formWindow() const; QWidget *integrationContainer(); void updateFormWindowSelectionHandles(bool state); -- GitLab