diff options
Diffstat (limited to 'gtk2_ardour')
-rw-r--r-- | gtk2_ardour/generic_pluginui.cc | 1 | ||||
-rw-r--r-- | gtk2_ardour/patch_change_dialog.cc | 15 | ||||
-rw-r--r-- | gtk2_ardour/patch_change_widget.cc | 3 |
3 files changed, 13 insertions, 6 deletions
diff --git a/gtk2_ardour/generic_pluginui.cc b/gtk2_ardour/generic_pluginui.cc index 1e48986a28..055129b704 100644 --- a/gtk2_ardour/generic_pluginui.cc +++ b/gtk2_ardour/generic_pluginui.cc @@ -732,7 +732,6 @@ GenericPluginUI::midi_refill_patches () for (MIDI::Name::PatchNameList::const_iterator j = patches.begin(); j != patches.end(); ++j) { const std::string pgm = (*j)->name (); MIDI::Name::PatchPrimaryKey const& key = (*j)->patch_primary_key (); - assert ((*i)->number () == key.bank()); const uint32_t bp = (key.bank() << 7) | key.program(); midi_pgmsel[chn]->AddMenuElem (MenuElemNoMnemonic (pgm, sigc::bind (sigc::mem_fun (*this, &GenericPluginUI::midi_bank_patch_select), chn, bp))); pgm_names[bp] = pgm; diff --git a/gtk2_ardour/patch_change_dialog.cc b/gtk2_ardour/patch_change_dialog.cc index 63aa9eac7b..961f91b512 100644 --- a/gtk2_ardour/patch_change_dialog.cc +++ b/gtk2_ardour/patch_change_dialog.cc @@ -116,9 +116,10 @@ PatchChangeDialog::PatchChangeDialog ( t->attach (_bank_lsb, 1, 2, r, r + 1); ++r; + assert (patch.bank() != UINT16_MAX); + _bank_msb.set_value ((patch.bank() >> 7)); _bank_msb.signal_changed().connect (sigc::mem_fun (*this, &PatchChangeDialog::bank_changed)); - _bank_lsb.set_value ((patch.bank() & 127)); _bank_lsb.signal_changed().connect (sigc::mem_fun (*this, &PatchChangeDialog::bank_changed)); @@ -270,10 +271,12 @@ PatchChangeDialog::bank_combo_changed () fill_patch_combo (); set_active_patch_combo (); - _ignore_signals = true; - _bank_msb.set_value (_current_patch_bank->number() >> 7); - _bank_lsb.set_value (_current_patch_bank->number() & 127); - _ignore_signals = false; + if (_current_patch_bank->number() != UINT16_MAX) { + _ignore_signals = true; + _bank_msb.set_value (_current_patch_bank->number() >> 7); + _bank_lsb.set_value (_current_patch_bank->number() & 127); + _ignore_signals = false; + } } /** Fill the contents of the patch combo */ @@ -345,6 +348,8 @@ PatchChangeDialog::patch_combo_changed () if (n == _patch_combo.get_active_text ()) { _ignore_signals = true; _program.set_value ((*j)->program_number() + 1); + _bank_msb.set_value ((*j)->bank_number() >> 7); + _bank_lsb.set_value ((*j)->bank_number() & 127); _ignore_signals = false; break; } diff --git a/gtk2_ardour/patch_change_widget.cc b/gtk2_ardour/patch_change_widget.cc index 922f7cd6f7..f9c88d8068 100644 --- a/gtk2_ardour/patch_change_widget.cc +++ b/gtk2_ardour/patch_change_widget.cc @@ -238,6 +238,9 @@ PatchChangeWidget::refill_banks () if (cns) { for (MIDI::Name::ChannelNameSet::PatchBanks::const_iterator i = cns->patch_banks().begin(); i != cns->patch_banks().end(); ++i) { std::string n = (*i)->name (); + if ((*i)->number () == UINT16_MAX) { + continue; + } _bank_select.AddMenuElem (MenuElemNoMnemonic (n, sigc::bind (sigc::mem_fun (*this, &PatchChangeWidget::select_bank), (*i)->number ()))); if ((*i)->number () == b) { _current_patch_bank = *i; |