diff options
author | Paul Davis <paul@linuxaudiosystems.com> | 2016-10-12 17:26:46 -0400 |
---|---|---|
committer | Paul Davis <paul@linuxaudiosystems.com> | 2016-10-12 17:26:46 -0400 |
commit | c104c9d4726f3ba1ecd352d13b88a57f2f964510 (patch) | |
tree | 61c1214d49f6efded889213e612d49fed5612927 /libs/ardour/phase_control.cc | |
parent | 27cff39dc7c8692e309ac0f9c515a583b775fd7d (diff) |
change return type of AutomationControl::actually_set_value() from void to bool, to indicate if value was changed.
Don't call Session::set_dirty() when no change occurs
Diffstat (limited to 'libs/ardour/phase_control.cc')
-rw-r--r-- | libs/ardour/phase_control.cc | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/libs/ardour/phase_control.cc b/libs/ardour/phase_control.cc index 9757c104b9..96d0d375b6 100644 --- a/libs/ardour/phase_control.cc +++ b/libs/ardour/phase_control.cc @@ -32,12 +32,12 @@ PhaseControl::PhaseControl (Session& session, std::string const & name) { } -void +bool PhaseControl::actually_set_value (double val, Controllable::GroupControlDisposition gcd) { _phase_invert = boost::dynamic_bitset<> (std::numeric_limits<double>::digits, (unsigned long) val); - AutomationControl::actually_set_value (val, gcd); + return AutomationControl::actually_set_value (val, gcd); } /** @param c Audio channel index. @@ -48,7 +48,7 @@ PhaseControl::set_phase_invert (uint32_t c, bool yn) { if (_phase_invert[c] != yn) { _phase_invert[c] = yn; - AutomationControl::actually_set_value (_phase_invert.to_ulong(), Controllable::NoGroup); + (void) AutomationControl::actually_set_value (_phase_invert.to_ulong(), Controllable::NoGroup); _session.set_dirty (); } } @@ -58,7 +58,7 @@ PhaseControl::set_phase_invert (boost::dynamic_bitset<> p) { if (_phase_invert != p) { _phase_invert = p; - AutomationControl::actually_set_value (_phase_invert.to_ulong(), Controllable::NoGroup); + (void) AutomationControl::actually_set_value (_phase_invert.to_ulong(), Controllable::NoGroup); Changed (true, Controllable::NoGroup); /* EMIT SIGNAL */ _session.set_dirty (); } |