summaryrefslogtreecommitdiff
path: root/libs/ardour/route_group.cc
diff options
context:
space:
mode:
authorPaul Davis <paul@linuxaudiosystems.com>2016-05-21 09:48:38 -0400
committerPaul Davis <paul@linuxaudiosystems.com>2016-05-31 15:30:44 -0400
commit2380bbae5bd77e39020ef779f1ae4a3255ca1e8a (patch)
tree3ad5dcfdc66010c97945e682a35cfbbfaa10bd61 /libs/ardour/route_group.cc
parenteaddf752c7106d33bf6730a591c10705a46ba172 (diff)
remove useless code recently added to RouteGroup
When a slaved automation control is changed because of a Master changing its own value, the slave notifies about the change with NoGroup as the GroupControlDisposition. There is therefore no need to disable a RouteGroup's enabled status for its ControlGroups - the changes initiated by any Master will not propagate to the group (which is precisely what we intend)
Diffstat (limited to 'libs/ardour/route_group.cc')
-rw-r--r--libs/ardour/route_group.cc41
1 files changed, 0 insertions, 41 deletions
diff --git a/libs/ardour/route_group.cc b/libs/ardour/route_group.cc
index a42f90a113..ecf4790d5c 100644
--- a/libs/ardour/route_group.cc
+++ b/libs/ardour/route_group.cc
@@ -295,10 +295,6 @@ RouteGroup::set_gain (bool yn)
_gain = yn;
_gain_group->set_active (yn);
- if (routes->empty() || !routes->front()->slaved()) {
- pre_master_gain = yn;
- }
-
send_change (PropertyChange (Properties::gain));
}
@@ -311,10 +307,6 @@ RouteGroup::set_mute (bool yn)
_mute = yn;
_mute_group->set_active (yn);
- if (routes->empty() || !routes->front()->slaved()) {
- pre_master_mute = yn;
- }
-
send_change (PropertyChange (Properties::mute));
}
@@ -327,10 +319,6 @@ RouteGroup::set_solo (bool yn)
_solo = yn;
_solo_group->set_active (yn);
- if (routes->empty() || !routes->front()->slaved()) {
- pre_master_solo = yn;
- }
-
send_change (PropertyChange (Properties::solo));
}
@@ -568,32 +556,13 @@ RouteGroup::assign_master (boost::shared_ptr<VCA> master)
boost::shared_ptr<Route> front = routes->front ();
- if (!front) {
- return;
- }
-
if (front->slaved_to (master)) {
return;
}
- bool cancel_master_controls = false;
-
- if (!front->slaved()) {
- pre_master_gain = is_gain ();
- pre_master_solo = is_solo ();
- pre_master_mute = is_mute ();
- cancel_master_controls = true;
- }
-
for (RouteList::iterator r = routes->begin(); r != routes->end(); ++r) {
(*r)->assign (master);
}
-
- if (cancel_master_controls) {
- set_gain (false);
- set_solo (false);
- set_mute (false);
- }
}
void
@@ -605,10 +574,6 @@ RouteGroup::unassign_master (boost::shared_ptr<VCA> master)
boost::shared_ptr<Route> front = routes->front ();
- if (!front) {
- return;
- }
-
if (!front->slaved_to (master)) {
return;
}
@@ -616,12 +581,6 @@ RouteGroup::unassign_master (boost::shared_ptr<VCA> master)
for (RouteList::iterator r = routes->begin(); r != routes->end(); ++r) {
(*r)->unassign (master);
}
-
- if (!front->slaved()) {
- set_gain (pre_master_gain);
- set_solo (pre_master_solo);
- set_mute (pre_master_mute);
- }
}
bool