Suppress compiler warnings about unused function parameters
[lxde/liblxqt.git] / configdialog / lxqtconfigdialog.cpp
index e469d94..947a578 100644 (file)
@@ -40,7 +40,6 @@ ConfigDialog::ConfigDialog(const QString& title, Settings* settings, QWidget* pa
     setWindowTitle(title);
     connect(ui->buttons, SIGNAL(clicked(QAbstractButton*)), SLOT(dialogButtonsAction(QAbstractButton*)));
     ui->moduleList->setVisible(false);
-    connect(Settings::globalSettings(), SIGNAL(settingsChanged()), this, SLOT(updateIcons()));
     foreach(QPushButton* button, ui->buttons->findChildren<QPushButton*>())
         button->setAutoDefault(false);
 }
@@ -59,6 +58,12 @@ void ConfigDialog::addPage(QWidget* page, const QString& name, const QString& ic
 
 void ConfigDialog::addPage(QWidget* page, const QString& name, const QStringList& iconNames)
 {
+    Q_ASSERT(page);
+    if (!page)
+    {
+        return;
+    }
+
     /* We set the layout margin to 0. In the default configuration, one page
      *  only, it aligns buttons with the page. In multi-page it saves a little
      *  bit of space, without clutter.
@@ -98,8 +103,16 @@ void ConfigDialog::showPage(QWidget* page)
     ui->moduleList->setCurrentRow(index);
 }
 
+bool ConfigDialog::event(QEvent * event)
+{
+    if (QEvent::ThemeChange == event->type())
+        updateIcons();
+    return QDialog::event(event);
+}
+
 void ConfigDialog::closeEvent(QCloseEvent* event)
 {
+    Q_UNUSED(event)
     emit save();
     mSettings->sync();
 }