From b0fe298f7295d1b076443c7252ecdb44ddb5896f Mon Sep 17 00:00:00 2001 From: Paul Davis Date: Tue, 1 Dec 2015 18:47:29 -0500 Subject: [PATCH] faderport: factor out encoder divider into a const double --- libs/surfaces/faderport/operations.cc | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/libs/surfaces/faderport/operations.cc b/libs/surfaces/faderport/operations.cc index 06eee19e96..0677c0adbb 100644 --- a/libs/surfaces/faderport/operations.cc +++ b/libs/surfaces/faderport/operations.cc @@ -30,6 +30,11 @@ using namespace ARDOUR; using namespace ArdourSurface; +/* this value is chosen to given smooth motion from 0..1.0 in about 270 degrees + * of encoder rotation. + */ +static const double encoder_divisor = 24.0; + void FaderPort::left () { @@ -226,7 +231,7 @@ FaderPort::ardour_pan_azimuth (int delta) return; } - azimuth->set_value (azimuth->interface_to_internal (azimuth->internal_to_interface (azimuth->get_value()) + (delta / 24.0))); + azimuth->set_value (azimuth->interface_to_internal (azimuth->internal_to_interface (azimuth->get_value()) + (delta / encoder_divider))); } @@ -249,7 +254,7 @@ FaderPort::ardour_pan_width(int delta) return; } - width->set_value (width->interface_to_internal (width->internal_to_interface (width->get_value()) + (delta / 24.0))); + width->set_value (width->interface_to_internal (width->internal_to_interface (width->get_value()) + (delta / encoder_divider))); } void