diff --git a/src/blackgui/pluginselector.cpp b/src/blackgui/pluginselector.cpp index b4f611f4e..39f580843 100644 --- a/src/blackgui/pluginselector.cpp +++ b/src/blackgui/pluginselector.cpp @@ -14,7 +14,6 @@ #include #include #include -#include #include #include #include @@ -22,9 +21,7 @@ namespace BlackGui { - CPluginSelector::CPluginSelector(QWidget *parent) : QWidget(parent), - m_detailsButtonMapper(new QSignalMapper(this)), - m_configButtonMapper(new QSignalMapper(this)) + CPluginSelector::CPluginSelector(QWidget *parent) : QWidget(parent) { setObjectName("CPluginSelector"); @@ -49,7 +46,7 @@ namespace BlackGui QCheckBox *cb = new QCheckBox(name); cb->setObjectName(identifier); cb->setProperty("pluginIdentifier", identifier); - connect(cb, &QCheckBox::stateChanged, this, &CPluginSelector::ps_handlePluginStateChange); + connect(cb, &QCheckBox::stateChanged, this, &CPluginSelector::handlePluginStateChange); if (enabled) { cb->setCheckState(Qt::Checked); @@ -88,14 +85,14 @@ namespace BlackGui cb->setChecked(enabled); } - void CPluginSelector::ps_handlePluginStateChange() + void CPluginSelector::handlePluginStateChange() { QCheckBox *cb = qobject_cast(sender()); Q_ASSERT(cb); bool enabled = cb->checkState() != Qt::Unchecked; Q_ASSERT(cb->property("pluginIdentifier").isValid()); - QString identifier = cb->property("pluginIdentifier").toString(); + const QString identifier = cb->property("pluginIdentifier").toString(); Q_ASSERT(!identifier.isEmpty()); emit pluginStateChanged(identifier, enabled); diff --git a/src/blackgui/pluginselector.h b/src/blackgui/pluginselector.h index acdfbbfa9..d72c3d0aa 100644 --- a/src/blackgui/pluginselector.h +++ b/src/blackgui/pluginselector.h @@ -17,8 +17,7 @@ #include #include #include - -class QSignalMapper; +#include namespace BlackGui { @@ -54,12 +53,12 @@ namespace BlackGui //! Enables/disabled the given plugin. void setEnabled(const QString &identifier, bool enabled); - private slots: - void ps_handlePluginStateChange(); private: - QSignalMapper *m_detailsButtonMapper; - QSignalMapper *m_configButtonMapper; + void handlePluginStateChange(); + + QSignalMapper *m_detailsButtonMapper = new QSignalMapper(this); + QSignalMapper *m_configButtonMapper = new QSignalMapper(this); }; } // ns