From a67e8630caf20e7f48151024e9e68dd9271d75c7 Mon Sep 17 00:00:00 2001 From: Stanislaw Halik Date: Fri, 1 Jul 2022 18:07:03 +0200 Subject: 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` instance. Also fix a bug in `options/tie.hpp` where `QComboBox::setCurrentIndex` was erroneously called as `Qt::DirectConnection`. --- spline/spline.cpp | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) (limited to 'spline') diff --git a/spline/spline.cpp b/spline/spline.cpp index ecbfc3a0..b8f84df5 100644 --- a/spline/spline.cpp +++ b/spline/spline.cpp @@ -390,12 +390,9 @@ void spline::set_bundle(bundle b, const QString& axis_name, Axis axis) invalidate_settings_(); S = s; - conn_points = QObject::connect(&s->points, value_::value_changed>(), - &*ctx, [this] { invalidate_settings(); }, Qt::DirectConnection); - conn_maxx = QObject::connect(&s->opts.clamp_x_, value_::value_changed(), - &*ctx, [this](double) { invalidate_settings(); }, Qt::DirectConnection); - conn_maxy = QObject::connect(&s->opts.clamp_y_, value_::value_changed(), - &*ctx, [this](double) { invalidate_settings(); }, Qt::DirectConnection); + conn_points = s->points.connect_to(&*ctx, &spline::invalidate_settings(), Qt::DirectConnection); + conn_maxx = s->opts.clamp_x_.connect_to(&*ctx, &spline::invalidate_settings, Qt::DirectConnection); + conn_maxy = s->opts.clamp_y_.connect_to(&*ctx, &spline::invalidate_settings(), Qt::DirectConnection); } emit S->recomputed(); -- cgit v1.2.3