ardour/ardour
ardour/ardour
13
1
Fork 0
Code Pull Requests Releases Activity

Don't offer possible recursive VCA assignments in the GUI

Browse Source
This commit is contained in:
Robin Gareus 2017-10-20 02:54:06 +02:00
parent bb4243c7a4
commit aea350c958
1 changed files with 5 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
gtk2_ardour/control_slave_ui.cc
View File

@ -191,22 +191,17 @@ ControlSlaveUI::vca_button_release (GdkEventButton* ev, uint32_t n)
for (VCAList::iterator v = vcas.begin(); v != vcas.end(); ++v) {
boost::shared_ptr<GainControl> gcs = stripable->gain_control();
boost::shared_ptr<GainControl> gcm = (*v)->gain_control();
if (gcs == gcm) {
/* asked to slave to self. not ok */
continue;
}
if (gcm->slaved_to (gcs)) {
/* master is already slaved to slave */
if (stripable->assigned_to (_session->vca_manager_ptr (), *v)) {
/* master(stripable) is directly or indirectly controlled by slave (v) */
continue;
}
items.push_back (CheckMenuElem ((*v)->name()));
Gtk::CheckMenuItem* item = dynamic_cast<Gtk::CheckMenuItem*> (&items.back());
boost::shared_ptr<GainControl> gcs = stripable->gain_control();
boost::shared_ptr<GainControl> gcm = (*v)->gain_control();
if (gcs->slaved_to (gcm)) {
item->set_active (true);
slaved = true;