options/value: add `QObject::connect` wrapper

This is useful not just to save on complexity in call sites, but also
because I plan on using the Verdigris library to remove needless
`valueChanged()` and `setValue()` overloads from each `value<t>`
instance.

Also fix a bug in `options/tie.hpp` where `QComboBox::setCurrentIndex`
was erroneously called as `Qt::DirectConnection`.

1 files changed, 2 insertions, 7 deletions

diff --git a/gui/options-dialog.cpp b/gui/options-dialog.cpp
index ac4117bc..ab36d73b 100644
--- a/gui/options-dialog.cpp
+++ b/gui/options-dialog.cpp
@@ -167,13 +167,8 @@ options_dialog::options_dialog(std::unique_ptr<ITrackerDialog>& tracker_dialog_,
     {
         tmp val = val_;
         val.label->setText(kopts_to_string(val.opt));
-        connect(&val.opt.keycode,
-                static_cast<void (value_::*)(const QString&) const>(&value_::valueChanged),
-                val.label,
-                [=](const QString&) { val.label->setText(kopts_to_string(val.opt)); });
-        {
-            connect(val.button, &QPushButton::clicked, this, [=] { bind_key(val.opt, val.label); });
-        }
+        val.opt.keycode.connect_to(val.label, [=] { val.label->setText(kopts_to_string(val.opt)); }, Qt::DirectConnection);
+        connect(val.button, &QPushButton::clicked, this, [=] { bind_key(val.opt, val.label); });
     }
 
     auto add_module_tab = [this] (auto& place, auto&& dlg, const QString& label) {