1 files changed, 10 insertions, 23 deletions

diff --git a/video-ps3eye/module.cpp b/video-ps3eye/module.cpp
index 812e523c..1d4e43f1 100644
--- a/video-ps3eye/module.cpp
+++ b/video-ps3eye/module.cpp
@@ -144,6 +144,7 @@ bool ps3eye_camera::start(info& args)
         return false;
 
     volatile auto& ptr = *(ps3eye::shm*)shm.ptr();
+    QString error;
 
     using mode = ps3eye::shm_in::mode;
 
@@ -181,14 +182,13 @@ bool ps3eye_camera::start(info& args)
         portable::sleep(sleep_ms);
     }
 
-    run_in_thread_sync(qApp, [&]() {
-        QString error;
-        if (ptr.out.error_string[0] == '\0')
-            error = "Unknown error";
-        else
-            error = QString::fromLatin1((const char*)ptr.out.error_string,
-                                        strnlen((const char*)ptr.out.error_string, sizeof(ptr.out.error_string)));
+    if (ptr.out.error_string[0] == '\0')
+        error = "Unknown error";
+    else
+        error = QString::fromLatin1((const char*)ptr.out.error_string,
+                                    strnlen((const char*)ptr.out.error_string, sizeof(ptr.out.error_string)));
 
+    run_in_thread_async(qApp, [=]() {
         QMessageBox::critical(nullptr, "Can't open camera", "PS3 Eye driver error: " + error, QMessageBox::Close);
     });
 
@@ -247,19 +247,16 @@ OTR_REGISTER_CAMERA(ps3eye_camera_)
 dialog::dialog(QWidget* parent) : QWidget(parent)
 {
     ui.setupUi(this);
-    t.setSingleShot(true);
-    t.setInterval(500);
     tie_setting(s.exposure, ui.exposure_slider);
     tie_setting(s.gain, ui.gain_slider);
     ui.exposure_label->setValue((int)*s.exposure);
     ui.gain_label->setValue((int)*s.gain);
-    connect(&s.exposure, value_::value_changed<slider_value>(), this, [this](const slider_value&) { s.set_exposure(); t.stop(); t.start(); });
-    connect(&s.gain, value_::value_changed<slider_value>(), this, [this](const slider_value&) { s.set_gain(); t.stop(); t.start(); });
+    connect(&s.exposure, value_::value_changed<slider_value>(), this, [this](const slider_value&) { s.set_exposure(); });
+    connect(&s.gain, value_::value_changed<slider_value>(), this, [this](const slider_value&) { s.set_gain(); });
     connect(ui.exposure_slider, &QSlider::valueChanged, ui.exposure_label, &QSpinBox::setValue);
     connect(ui.gain_slider, &QSlider::valueChanged, ui.gain_label, &QSpinBox::setValue);
     connect(ui.buttonBox, &QDialogButtonBox::accepted, this, &dialog::do_ok);
     connect(ui.buttonBox, &QDialogButtonBox::rejected, this, &dialog::do_cancel);
-    connect(&t, &QTimer::timeout, this, [this]() { s.apply(); });
 }
 
 // XXX copypasta -sh 20200329
@@ -270,7 +267,7 @@ void settings::set_gain()
 
     auto& ptr = *(ps3eye::shm volatile*)shm.ptr();
     ptr.in.gain = (unsigned char)*gain;
-    //++ptr.in.settings_updated;
+    ++ptr.in.settings_updated;
     std::atomic_thread_fence(std::memory_order_seq_cst);
 }
 
@@ -281,16 +278,6 @@ void settings::set_exposure()
 
     auto& ptr = *(ps3eye::shm volatile*)shm.ptr();
     ptr.in.exposure = (unsigned char)*exposure;
-    //++ptr.in.settings_updated;
-    std::atomic_thread_fence(std::memory_order_seq_cst);
-}
-
-void settings::apply()
-{
-    if (!shm.success())
-        return;
-
-    auto& ptr = *(ps3eye::shm volatile*)shm.ptr();
     ++ptr.in.settings_updated;
     std::atomic_thread_fence(std::memory_order_seq_cst);
 }