diff options
author | Friedemann Kleint <Friedemann.Kleint@theqtcompany.com> | 2015-04-10 11:42:18 +0200 |
---|---|---|
committer | Friedemann Kleint <Friedemann.Kleint@theqtcompany.com> | 2015-04-10 10:34:06 +0000 |
commit | 6d0c86d65b6fa0f37ba696de28fea4e977d0a282 (patch) | |
tree | 10a0c5eb676c27143db4b1ce36ad012d375534b2 | |
parent | 1b04bf6d36af200dff0d7afff7080afab278034a (diff) |
Qt Designer Examples: Fix code around constructors.
Make constructors explicit, use constructor initialization for member
variables, remove unused variables.
Change-Id: I84273400d66ef061376bd0ec4948ab1a0fd45f30
Reviewed-by: Jarek Kobus <jaroslaw.kobus@theqtcompany.com>
20 files changed, 31 insertions, 35 deletions
diff --git a/examples/designer/containerextension/multipagewidget.cpp b/examples/designer/containerextension/multipagewidget.cpp index 767397a94..4d9e64713 100644 --- a/examples/designer/containerextension/multipagewidget.cpp +++ b/examples/designer/containerextension/multipagewidget.cpp @@ -46,20 +46,19 @@ MultiPageWidget::MultiPageWidget(QWidget *parent) : QWidget(parent) + , stackWidget(new QStackedWidget) + , comboBox(new QComboBox) { typedef void (QComboBox::*ComboBoxActivatedIntSignal)(int); - comboBox = new QComboBox(); comboBox->setObjectName("__qt__passive_comboBox"); - stackWidget = new QStackedWidget(); connect(comboBox, static_cast<ComboBoxActivatedIntSignal>(&QComboBox::activated), this, &MultiPageWidget::setCurrentIndex); - layout = new QVBoxLayout(); + QVBoxLayout *layout = new QVBoxLayout(this); layout->addWidget(comboBox); layout->addWidget(stackWidget); - setLayout(layout); } QSize MultiPageWidget::sizeHint() const diff --git a/examples/designer/containerextension/multipagewidget.h b/examples/designer/containerextension/multipagewidget.h index 850111515..4b726d774 100644 --- a/examples/designer/containerextension/multipagewidget.h +++ b/examples/designer/containerextension/multipagewidget.h @@ -46,7 +46,6 @@ QT_BEGIN_NAMESPACE class QComboBox; class QStackedWidget; -class QVBoxLayout; QT_END_NAMESPACE //! [0] @@ -57,7 +56,7 @@ class MultiPageWidget : public QWidget Q_PROPERTY(QString pageTitle READ pageTitle WRITE setPageTitle STORED false) public: - MultiPageWidget(QWidget *parent = 0); + explicit MultiPageWidget(QWidget *parent = 0); QSize sizeHint() const Q_DECL_OVERRIDE; @@ -83,7 +82,6 @@ signals: private: QStackedWidget *stackWidget; QComboBox *comboBox; - QVBoxLayout *layout; }; //! [0] diff --git a/examples/designer/containerextension/multipagewidgetcontainerextension.cpp b/examples/designer/containerextension/multipagewidgetcontainerextension.cpp index 29efd459a..b06b0181a 100644 --- a/examples/designer/containerextension/multipagewidgetcontainerextension.cpp +++ b/examples/designer/containerextension/multipagewidgetcontainerextension.cpp @@ -44,9 +44,9 @@ //! [0] MultiPageWidgetContainerExtension::MultiPageWidgetContainerExtension(MultiPageWidget *widget, QObject *parent) - :QObject(parent) + : QObject(parent) + , myWidget(widget) { - myWidget = widget; } //! [0] diff --git a/examples/designer/containerextension/multipagewidgetcontainerextension.h b/examples/designer/containerextension/multipagewidgetcontainerextension.h index 1dcc88c15..f7cd8f1fe 100644 --- a/examples/designer/containerextension/multipagewidgetcontainerextension.h +++ b/examples/designer/containerextension/multipagewidgetcontainerextension.h @@ -56,7 +56,7 @@ class MultiPageWidgetContainerExtension: public QObject, Q_INTERFACES(QDesignerContainerExtension) public: - MultiPageWidgetContainerExtension(MultiPageWidget *widget, QObject *parent); + explicit MultiPageWidgetContainerExtension(MultiPageWidget *widget, QObject *parent); void addWidget(QWidget *widget) Q_DECL_OVERRIDE; int count() const Q_DECL_OVERRIDE; diff --git a/examples/designer/containerextension/multipagewidgetextensionfactory.h b/examples/designer/containerextension/multipagewidgetextensionfactory.h index 2ece4e148..ace3bcdf3 100644 --- a/examples/designer/containerextension/multipagewidgetextensionfactory.h +++ b/examples/designer/containerextension/multipagewidgetextensionfactory.h @@ -53,7 +53,7 @@ class MultiPageWidgetExtensionFactory: public QExtensionFactory Q_OBJECT public: - MultiPageWidgetExtensionFactory(QExtensionManager *parent = 0); + explicit MultiPageWidgetExtensionFactory(QExtensionManager *parent = 0); protected: QObject *createExtension(QObject *object, const QString &iid, QObject *parent) const Q_DECL_OVERRIDE; diff --git a/examples/designer/containerextension/multipagewidgetplugin.cpp b/examples/designer/containerextension/multipagewidgetplugin.cpp index 950f2ed04..4e858a63a 100644 --- a/examples/designer/containerextension/multipagewidgetplugin.cpp +++ b/examples/designer/containerextension/multipagewidgetplugin.cpp @@ -54,9 +54,9 @@ //! [0] MultiPageWidgetPlugin::MultiPageWidgetPlugin(QObject *parent) - :QObject(parent) + : QObject(parent) + , initialized(false) { - initialized = false; } QString MultiPageWidgetPlugin::name() const diff --git a/examples/designer/containerextension/multipagewidgetplugin.h b/examples/designer/containerextension/multipagewidgetplugin.h index 2099819fe..34a183668 100644 --- a/examples/designer/containerextension/multipagewidgetplugin.h +++ b/examples/designer/containerextension/multipagewidgetplugin.h @@ -57,7 +57,7 @@ class MultiPageWidgetPlugin: public QObject, public QDesignerCustomWidgetInterfa //! [1] Q_INTERFACES(QDesignerCustomWidgetInterface) public: - MultiPageWidgetPlugin(QObject *parent = 0); + explicit MultiPageWidgetPlugin(QObject *parent = 0); QString name() const Q_DECL_OVERRIDE; QString group() const Q_DECL_OVERRIDE; diff --git a/examples/designer/customwidgetplugin/analogclock.h b/examples/designer/customwidgetplugin/analogclock.h index fd4056c05..1ea6d9eb4 100644 --- a/examples/designer/customwidgetplugin/analogclock.h +++ b/examples/designer/customwidgetplugin/analogclock.h @@ -49,7 +49,7 @@ class QDESIGNER_WIDGET_EXPORT AnalogClock : public QWidget Q_OBJECT public: - AnalogClock(QWidget *parent = 0); + explicit AnalogClock(QWidget *parent = 0); protected: void paintEvent(QPaintEvent *event) Q_DECL_OVERRIDE; diff --git a/examples/designer/customwidgetplugin/customwidgetplugin.cpp b/examples/designer/customwidgetplugin/customwidgetplugin.cpp index eeb0a8499..67b9c28dd 100644 --- a/examples/designer/customwidgetplugin/customwidgetplugin.cpp +++ b/examples/designer/customwidgetplugin/customwidgetplugin.cpp @@ -46,8 +46,8 @@ //! [0] AnalogClockPlugin::AnalogClockPlugin(QObject *parent) : QObject(parent) + , initialized(false) { - initialized = false; } //! [0] diff --git a/examples/designer/customwidgetplugin/customwidgetplugin.h b/examples/designer/customwidgetplugin/customwidgetplugin.h index d1a09ed7d..a5037477d 100644 --- a/examples/designer/customwidgetplugin/customwidgetplugin.h +++ b/examples/designer/customwidgetplugin/customwidgetplugin.h @@ -50,7 +50,7 @@ class AnalogClockPlugin : public QObject, public QDesignerCustomWidgetInterface Q_PLUGIN_METADATA(IID "org.qt-project.Qt.QDesignerCustomWidgetInterface" FILE "analogclock.json") Q_INTERFACES(QDesignerCustomWidgetInterface) public: - AnalogClockPlugin(QObject *parent = 0); + explicit AnalogClockPlugin(QObject *parent = 0); bool isContainer() const Q_DECL_OVERRIDE; bool isInitialized() const Q_DECL_OVERRIDE; diff --git a/examples/designer/taskmenuextension/tictactoe.cpp b/examples/designer/taskmenuextension/tictactoe.cpp index b79beeeac..c46acb0c6 100644 --- a/examples/designer/taskmenuextension/tictactoe.cpp +++ b/examples/designer/taskmenuextension/tictactoe.cpp @@ -45,6 +45,7 @@ TicTacToe::TicTacToe(QWidget *parent) : QWidget(parent) + , turnNumber(0) { } diff --git a/examples/designer/taskmenuextension/tictactoe.h b/examples/designer/taskmenuextension/tictactoe.h index 14a6b6a9d..d31e8e446 100644 --- a/examples/designer/taskmenuextension/tictactoe.h +++ b/examples/designer/taskmenuextension/tictactoe.h @@ -55,7 +55,7 @@ class TicTacToe : public QWidget Q_PROPERTY(QString state READ state WRITE setState) public: - TicTacToe(QWidget *parent = 0); + explicit TicTacToe(QWidget *parent = 0); QSize minimumSizeHint() const Q_DECL_OVERRIDE; QSize sizeHint() const Q_DECL_OVERRIDE; diff --git a/examples/designer/taskmenuextension/tictactoedialog.cpp b/examples/designer/taskmenuextension/tictactoedialog.cpp index e6d1d415e..6c6e2f3d7 100644 --- a/examples/designer/taskmenuextension/tictactoedialog.cpp +++ b/examples/designer/taskmenuextension/tictactoedialog.cpp @@ -51,15 +51,14 @@ //! [0] TicTacToeDialog::TicTacToeDialog(TicTacToe *tic, QWidget *parent) : QDialog(parent) + , editor(new TicTacToe) + , ticTacToe(tic) + , buttonBox(new QDialogButtonBox(QDialogButtonBox::Ok + | QDialogButtonBox::Cancel + | QDialogButtonBox::Reset)) { - ticTacToe = tic; - editor = new TicTacToe; editor->setState(ticTacToe->state()); - buttonBox = new QDialogButtonBox(QDialogButtonBox::Ok - | QDialogButtonBox::Cancel - | QDialogButtonBox::Reset); - connect(buttonBox->button(QDialogButtonBox::Reset), &QAbstractButton::clicked, this, &TicTacToeDialog::resetState); connect(buttonBox, &QDialogButtonBox::accepted, this, &TicTacToeDialog::saveState); diff --git a/examples/designer/taskmenuextension/tictactoeplugin.cpp b/examples/designer/taskmenuextension/tictactoeplugin.cpp index 2c953f678..770a44a24 100644 --- a/examples/designer/taskmenuextension/tictactoeplugin.cpp +++ b/examples/designer/taskmenuextension/tictactoeplugin.cpp @@ -49,8 +49,8 @@ //! [0] TicTacToePlugin::TicTacToePlugin(QObject *parent) : QObject(parent) + , initialized(false) { - initialized = false; } QString TicTacToePlugin::name() const diff --git a/examples/designer/taskmenuextension/tictactoetaskmenu.cpp b/examples/designer/taskmenuextension/tictactoetaskmenu.cpp index 961d5a1d4..e71d96c4c 100644 --- a/examples/designer/taskmenuextension/tictactoetaskmenu.cpp +++ b/examples/designer/taskmenuextension/tictactoetaskmenu.cpp @@ -48,10 +48,9 @@ //! [0] TicTacToeTaskMenu::TicTacToeTaskMenu(TicTacToe *tic, QObject *parent) : QObject(parent) + , editStateAction(new QAction(tr("Edit State..."), this)) + , ticTacToe(tic) { - ticTacToe = tic; - - editStateAction = new QAction(tr("Edit State..."), this); connect(editStateAction, &QAction::triggered, this, &TicTacToeTaskMenu::editState); } //! [0] diff --git a/examples/designer/taskmenuextension/tictactoetaskmenu.h b/examples/designer/taskmenuextension/tictactoetaskmenu.h index 0f41cf8ba..37577896f 100644 --- a/examples/designer/taskmenuextension/tictactoetaskmenu.h +++ b/examples/designer/taskmenuextension/tictactoetaskmenu.h @@ -57,7 +57,7 @@ class TicTacToeTaskMenu : public QObject, public QDesignerTaskMenuExtension Q_INTERFACES(QDesignerTaskMenuExtension) public: - TicTacToeTaskMenu(TicTacToe *tic, QObject *parent); + explicit TicTacToeTaskMenu(TicTacToe *tic, QObject *parent); QAction *preferredEditAction() const Q_DECL_OVERRIDE; QList<QAction *> taskActions() const Q_DECL_OVERRIDE; @@ -77,7 +77,7 @@ class TicTacToeTaskMenuFactory : public QExtensionFactory Q_OBJECT public: - TicTacToeTaskMenuFactory(QExtensionManager *parent = 0); + explicit TicTacToeTaskMenuFactory(QExtensionManager *parent = 0); protected: QObject *createExtension(QObject *object, const QString &iid, QObject *parent) const Q_DECL_OVERRIDE; diff --git a/examples/designer/worldtimeclockplugin/worldtimeclock.cpp b/examples/designer/worldtimeclockplugin/worldtimeclock.cpp index 1a4d8fcda..776eec786 100644 --- a/examples/designer/worldtimeclockplugin/worldtimeclock.cpp +++ b/examples/designer/worldtimeclockplugin/worldtimeclock.cpp @@ -46,11 +46,11 @@ WorldTimeClock::WorldTimeClock(QWidget *parent) : QWidget(parent) + , timeZoneOffset(0) + { typedef void (QWidget::*WidgetUpdateSlot)(); - timeZoneOffset = 0; - QTimer *timer = new QTimer(this); connect(timer, &QTimer::timeout, this, static_cast<WidgetUpdateSlot>(&QWidget::update)); timer->start(1000); diff --git a/examples/designer/worldtimeclockplugin/worldtimeclock.h b/examples/designer/worldtimeclockplugin/worldtimeclock.h index 497c51b4a..e0220a8dc 100644 --- a/examples/designer/worldtimeclockplugin/worldtimeclock.h +++ b/examples/designer/worldtimeclockplugin/worldtimeclock.h @@ -52,7 +52,7 @@ class QDESIGNER_WIDGET_EXPORT WorldTimeClock : public QWidget //! [0] public: - WorldTimeClock(QWidget *parent = 0); + explicit WorldTimeClock(QWidget *parent = 0); public slots: void setTimeZone(int hourOffset); diff --git a/examples/designer/worldtimeclockplugin/worldtimeclockplugin.cpp b/examples/designer/worldtimeclockplugin/worldtimeclockplugin.cpp index 6c879fdfb..416c8288e 100644 --- a/examples/designer/worldtimeclockplugin/worldtimeclockplugin.cpp +++ b/examples/designer/worldtimeclockplugin/worldtimeclockplugin.cpp @@ -45,8 +45,8 @@ WorldTimeClockPlugin::WorldTimeClockPlugin(QObject *parent) : QObject(parent) + , initialized(false) { - initialized = false; } void WorldTimeClockPlugin::initialize(QDesignerFormEditorInterface * /* core */) diff --git a/examples/designer/worldtimeclockplugin/worldtimeclockplugin.h b/examples/designer/worldtimeclockplugin/worldtimeclockplugin.h index a4bb24223..7ed81ba6f 100644 --- a/examples/designer/worldtimeclockplugin/worldtimeclockplugin.h +++ b/examples/designer/worldtimeclockplugin/worldtimeclockplugin.h @@ -54,7 +54,7 @@ class WorldTimeClockPlugin : public QObject, Q_INTERFACES(QDesignerCustomWidgetInterface) public: - WorldTimeClockPlugin(QObject *parent = 0); + explicit WorldTimeClockPlugin(QObject *parent = 0); bool isContainer() const Q_DECL_OVERRIDE; bool isInitialized() const Q_DECL_OVERRIDE; |