Convert polling rate slot to lambda

Easier to read and also a bit less code.
z3ntu · May 4, 2024 · fc7513c · fc7513c
1 parent aa2058d
commit fc7513c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 15 deletions.
diff --git a/src/devicewidget/performancewidget.cpp b/src/devicewidget/performancewidget.cpp
@@ -57,7 +57,14 @@ PerformanceWidget::PerformanceWidget(libopenrazer::Device *device)
         pollComboBox->setCurrentText(QString::number(pollRate) + " Hz");
         verticalLayout->addWidget(pollComboBox);
 
-        connect(pollComboBox, QOverload<int>::of(&QComboBox::currentIndexChanged), this, &PerformanceWidget::pollCombo);
+        connect(pollComboBox, QOverload<int>::of(&QComboBox::currentIndexChanged), this, [=](int) {
+            try {
+                device->setPollRate(pollComboBox->currentData().value<ushort>());
+            } catch (const libopenrazer::DBusException &e) {
+                qWarning("Failed to set polling rate");
+                util::showError(tr("Failed to set polling rate"));
+            }
+        });
     }
 
     /* Spacer to bottom */
@@ -71,14 +78,3 @@ bool PerformanceWidget::isAvailable(libopenrazer::Device *device)
 {
     return device->hasFeature("dpi") || device->hasFeature("poll_rate");
 }
-
-void PerformanceWidget::pollCombo(int /* index */)
-{
-    auto *sender = qobject_cast<QComboBox *>(QObject::sender());
-    try {
-        device->setPollRate(sender->currentData().value<ushort>());
-    } catch (const libopenrazer::DBusException &e) {
-        qWarning("Failed to set polling rate");
-        util::showError(tr("Failed to set polling rate"));
-    }
-}
diff --git a/src/devicewidget/performancewidget.h b/src/devicewidget/performancewidget.h
@@ -17,9 +17,6 @@ class PerformanceWidget : public QWidget
 
     static bool isAvailable(libopenrazer::Device *device);
 
-public slots:
-    void pollCombo(int /* index */);
-
 private:
     libopenrazer::Device *device;
 };