From d2cafbe95a5784b7c306c24b0008379a41a909f7 Mon Sep 17 00:00:00 2001 From: David Robillard Date: Wed, 17 Dec 2014 19:43:09 -0500 Subject: Remove some aborts that don't really need to be. Enforce PatchPrimaryKey sanity at the type level rather than attempting to check for it everywhere. Remove dead file. --- gtk2_ardour/automation_streamview.cc | 22 ++- gtk2_ardour/canvas_patch_change.cc | 253 ----------------------------------- gtk2_ardour/midi_region_view.cc | 67 ++-------- gtk2_ardour/midi_region_view.h | 12 +- gtk2_ardour/patch_change.cc | 8 +- gtk2_ardour/patch_change_dialog.cc | 2 +- gtk2_ardour/region_view.cc | 1 - 7 files changed, 36 insertions(+), 329 deletions(-) delete mode 100644 gtk2_ardour/canvas_patch_change.cc (limited to 'gtk2_ardour') diff --git a/gtk2_ardour/automation_streamview.cc b/gtk2_ardour/automation_streamview.cc index e504d13931..a336bd6db5 100644 --- a/gtk2_ardour/automation_streamview.cc +++ b/gtk2_ardour/automation_streamview.cc @@ -16,7 +16,6 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. */ -#include #include #include #include @@ -43,6 +42,8 @@ #include "rgb_macros.h" #include "selection.h" +#include "i18n.h" + using namespace std; using namespace ARDOUR; using namespace ARDOUR_UI_UTILS; @@ -71,7 +72,9 @@ AutomationStreamView::~AutomationStreamView () RegionView* AutomationStreamView::add_region_view_internal (boost::shared_ptr region, bool wait_for_data, bool /*recording*/) { - assert (region); + if (!region) { + return 0; + } if (wait_for_data) { boost::shared_ptr mr = boost::dynamic_pointer_cast(region); @@ -88,7 +91,10 @@ AutomationStreamView::add_region_view_internal (boost::shared_ptr region boost::shared_ptr list; if (control) { list = boost::dynamic_pointer_cast(control->list()); - assert(!control->list() || list); + if (control->list() && !list) { + error << _("unable to display automation region for control without list") << endmsg; + return 0; + } } AutomationRegionView *region_view; @@ -285,8 +291,9 @@ AutomationStreamView::get_selectables (framepos_t start, framepos_t end, double } for (list::iterator i = region_views.begin(); i != region_views.end(); ++i) { AutomationRegionView* arv = dynamic_cast (*i); - assert (arv); - arv->line()->get_selectables (start, end, botfrac, topfrac, results); + if (arv) { + arv->line()->get_selectables (start, end, botfrac, topfrac, results); + } } } @@ -307,8 +314,9 @@ AutomationStreamView::get_lines () const for (list::const_iterator i = region_views.begin(); i != region_views.end(); ++i) { AutomationRegionView* arv = dynamic_cast (*i); - assert (arv); - lines.push_back (arv->line()); + if (arv) { + lines.push_back (arv->line()); + } } return lines; diff --git a/gtk2_ardour/canvas_patch_change.cc b/gtk2_ardour/canvas_patch_change.cc deleted file mode 100644 index 68dbbe3b0d..0000000000 --- a/gtk2_ardour/canvas_patch_change.cc +++ /dev/null @@ -1,253 +0,0 @@ -/* - Copyright (C) 2000-2010 Paul Davis - - This program is free software; you can redistribute it and/or modify - it under the terms of the GNU General Public License as published by - the Free Software Foundation; either version 2 of the License, or - (at your option) any later version. - - This program is distributed in the hope that it will be useful, - but WITHOUT ANY WARRANTY; without even the implied warranty of - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - GNU General Public License for more details. - - You should have received a copy of the GNU General Public License - along with this program; if not, write to the Free Software - Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. - -*/ - -#include - -#include - -#include "pbd/stacktrace.h" - -#include "gtkmm2ext/keyboard.h" -#include "ardour/instrument_info.h" -#include "midi++/midnam_patch.h" - -#include "ardour_ui.h" -#include "canvas_patch_change.h" -#include "editor.h" -#include "editor_drag.h" - -using namespace Gnome::Canvas; -using namespace MIDI::Name; -using namespace Gtkmm2ext; -using namespace std; - -/** @param x x position in pixels. - */ -CanvasPatchChange::CanvasPatchChange( - MidiRegionView& region, - Group& parent, - const string& text, - double height, - double x, - double y, - ARDOUR::InstrumentInfo& info, - ARDOUR::MidiModel::PatchChangePtr patch, - bool active_channel) - : CanvasFlag( - region, - parent, - height, - (active_channel - ? ARDOUR_UI::config()->get_MidiPatchChangeOutline() - : ARDOUR_UI::config()->get_MidiPatchChangeInactiveChannelOutline()), - (active_channel - ? ARDOUR_UI::config()->get_MidiPatchChangeFill() - : ARDOUR_UI::config()->get_MidiPatchChangeInactiveChannelFill()), - x, - y) - , _info (info) - , _patch (patch) - , _popup_initialized(false) -{ - set_text (text); -} - -CanvasPatchChange::~CanvasPatchChange() -{ -} - -void -CanvasPatchChange::initialize_popup_menus() -{ - boost::shared_ptr channel_name_set = _info.get_patches (_patch->channel()); - - if (!channel_name_set) { - return; - } - - const ChannelNameSet::PatchBanks& patch_banks = channel_name_set->patch_banks(); - - if (patch_banks.size() > 1) { - // fill popup menu: - Gtk::Menu::MenuList& patch_bank_menus = _popup.items(); - - for (ChannelNameSet::PatchBanks::const_iterator bank = patch_banks.begin(); - bank != patch_banks.end(); - ++bank) { - Gtk::Menu& patch_bank_menu = *manage(new Gtk::Menu()); - - const PatchNameList& patches = (*bank)->patch_name_list(); - Gtk::Menu::MenuList& patch_menus = patch_bank_menu.items(); - - for (PatchNameList::const_iterator patch = patches.begin(); - patch != patches.end(); - ++patch) { - std::string name = (*patch)->name(); - boost::replace_all (name, "_", " "); - - patch_menus.push_back( - Gtk::Menu_Helpers::MenuElem( - name, - sigc::bind(sigc::mem_fun(*this, &CanvasPatchChange::on_patch_menu_selected), - (*patch)->patch_primary_key())) ); - } - - std::string name = (*bank)->name(); - boost::replace_all (name, "_", " "); - - patch_bank_menus.push_back( - Gtk::Menu_Helpers::MenuElem( - name, - patch_bank_menu) ); - } - } else { - /* only one patch bank, so make it the initial menu */ - - const PatchNameList& patches = patch_banks.front()->patch_name_list(); - Gtk::Menu::MenuList& patch_menus = _popup.items(); - - for (PatchNameList::const_iterator patch = patches.begin(); - patch != patches.end(); - ++patch) { - std::string name = (*patch)->name(); - boost::replace_all (name, "_", " "); - - patch_menus.push_back ( - Gtk::Menu_Helpers::MenuElem ( - name, - sigc::bind (sigc::mem_fun(*this, &CanvasPatchChange::on_patch_menu_selected), - (*patch)->patch_primary_key()))); - } - } -} - -void -CanvasPatchChange::on_patch_menu_selected(const PatchPrimaryKey& key) -{ - _region.change_patch_change (*this, key); -} - -static bool -in_edit_mode(Editor* editor) -{ - return (editor->internal_editing() && - (editor->current_mouse_mode() == Editing::MouseObject || - editor->current_mouse_mode() == Editing::MouseDraw)); -} - -bool -CanvasPatchChange::on_event (GdkEvent* ev) -{ - /* XXX: icky dcast */ - Editor* e = dynamic_cast (&_region.get_time_axis_view().editor()); - - if (!in_edit_mode(e)) { - return false; - } - - switch (ev->type) { - case GDK_BUTTON_PRESS: - if (Gtkmm2ext::Keyboard::is_delete_event (&ev->button)) { - - _region.delete_patch_change (this); - return true; - - } else if (Gtkmm2ext::Keyboard::is_edit_event (&ev->button)) { - - _region.edit_patch_change (this); - return true; - - } else if (ev->button.button == 1) { - e->drags()->set (new PatchChangeDrag (e, this, &_region), ev); - return true; - } - - if (ev->button.button == 3) { - if (!_popup_initialized) { - initialize_popup_menus(); - _popup_initialized = true; - } - if (!_popup.items().empty()) { - _popup.popup(ev->button.button, ev->button.time); - } - return true; - } - break; - - case GDK_KEY_PRESS: - switch (ev->key.keyval) { - case GDK_Up: - case GDK_KP_Up: - case GDK_uparrow: - if (Keyboard::modifier_state_contains (ev->key.state, Keyboard::PrimaryModifier)) { - _region.previous_bank (*this); - } else { - _region.previous_patch (*this); - } - break; - case GDK_Down: - case GDK_KP_Down: - case GDK_downarrow: - if (Keyboard::modifier_state_contains (ev->key.state, Keyboard::PrimaryModifier)) { - _region.next_bank (*this); - } else { - _region.next_patch (*this); - } - break; - case GDK_Delete: - case GDK_BackSpace: - _region.delete_patch_change (this); - break; - default: - break; - } - break; - - case GDK_SCROLL: - if (ev->scroll.direction == GDK_SCROLL_UP) { - if (Keyboard::modifier_state_contains (ev->scroll.state, Keyboard::PrimaryModifier)) { - _region.previous_bank (*this); - } else { - _region.previous_patch (*this); - } - } else if (ev->scroll.direction == GDK_SCROLL_DOWN) { - if (Keyboard::modifier_state_contains (ev->scroll.state, Keyboard::PrimaryModifier)) { - _region.next_bank (*this); - } else { - _region.next_patch (*this); - } - } - return true; - - case GDK_ENTER_NOTIFY: - _region.patch_entered (this); - return true; - break; - - case GDK_LEAVE_NOTIFY: - _region.patch_left (this); - return true; - break; - - default: - break; - } - - return false; -} diff --git a/gtk2_ardour/midi_region_view.cc b/gtk2_ardour/midi_region_view.cc index 2879e65af7..11534dd035 100644 --- a/gtk2_ardour/midi_region_view.cc +++ b/gtk2_ardour/midi_region_view.cc @@ -1881,15 +1881,11 @@ MidiRegionView::get_patch_key_at (Evoral::MusicalTime time, uint8_t channel, MID } if (i != _model->patch_changes().end() && patch_applies(*i, time, channel)) { - key.bank_number = (*i)->bank(); - key.program_number = (*i)->program (); + key.set_bank((*i)->bank()); + key.set_program((*i)->program ()); } else { - key.bank_number = key.program_number = 0; - } - - if (!key.is_sane()) { - error << string_compose(_("insane MIDI patch key %1:%2"), - key.bank_number, key.program_number) << endmsg; + key.set_bank(0); + key.set_program(0); } } @@ -1898,11 +1894,11 @@ MidiRegionView::change_patch_change (PatchChange& pc, const MIDI::Name::PatchPri { MidiModel::PatchChangeDiffCommand* c = _model->new_patch_change_diff_command (_("alter patch change")); - if (pc.patch()->program() != new_patch.program_number) { - c->change_program (pc.patch (), new_patch.program_number); + if (pc.patch()->program() != new_patch.program()) { + c->change_program (pc.patch (), new_patch.program()); } - int const new_bank = new_patch.bank_number; + int const new_bank = new_patch.bank(); if (pc.patch()->bank() != new_bank) { c->change_bank (pc.patch (), new_bank); } @@ -1988,47 +1984,12 @@ MidiRegionView::delete_patch_change (PatchChange* pc) } void -MidiRegionView::previous_patch (PatchChange& patch) -{ - if (patch.patch()->program() < 127) { - MIDI::Name::PatchPrimaryKey key = patch_change_to_patch_key (patch.patch()); - key.program_number++; - change_patch_change (patch, key); - } -} - -void -MidiRegionView::next_patch (PatchChange& patch) -{ - if (patch.patch()->program() > 0) { - MIDI::Name::PatchPrimaryKey key = patch_change_to_patch_key (patch.patch()); - key.program_number--; - change_patch_change (patch, key); - } -} - -void -MidiRegionView::next_bank (PatchChange& patch) -{ - if (patch.patch()->program() < 127) { - MIDI::Name::PatchPrimaryKey key = patch_change_to_patch_key (patch.patch()); - if (key.bank_number > 0) { - key.bank_number--; - change_patch_change (patch, key); - } - } -} - -void -MidiRegionView::previous_bank (PatchChange& patch) +MidiRegionView::step_patch (PatchChange& patch, int dbank, int dprog) { - if (patch.patch()->program() > 0) { - MIDI::Name::PatchPrimaryKey key = patch_change_to_patch_key (patch.patch()); - if (key.bank_number < 127) { - key.bank_number++; - change_patch_change (patch, key); - } - } + MIDI::Name::PatchPrimaryKey key = patch_change_to_patch_key(patch.patch()); + key.set_bank(key.bank() + dbank); + key.set_program(key.program() + dprog); + change_patch_change(patch, key); } void @@ -3893,8 +3854,8 @@ MidiRegionView::show_verbose_cursor (boost::shared_ptr n) const get_patch_key_at(n->time(), n->channel(), patch_key); name = device_names->note_name(mtv->gui_property(X_("midnam-custom-device-mode")), n->channel(), - patch_key.bank_number, - patch_key.program_number, + patch_key.bank(), + patch_key.program(), n->note()); } } diff --git a/gtk2_ardour/midi_region_view.h b/gtk2_ardour/midi_region_view.h index a2d364c0d6..a07a46825f 100644 --- a/gtk2_ardour/midi_region_view.h +++ b/gtk2_ardour/midi_region_view.h @@ -153,16 +153,8 @@ public: void delete_sysex (SysEx*); - /** Alter a given patch to be its predecessor in the MIDNAM file. - */ - void previous_patch (PatchChange &); - - /** Alters a given patch to be its successor in the MIDNAM file. - */ - void next_patch (PatchChange &); - - void previous_bank (PatchChange &); - void next_bank (PatchChange &); + /** Change a patch to the next or previous bank/program. */ + void step_patch (PatchChange& patch, int dbank, int dprog); /** Displays all patch change events in the region as flags on the canvas. */ diff --git a/gtk2_ardour/patch_change.cc b/gtk2_ardour/patch_change.cc index 1701e66b56..844a129b1d 100644 --- a/gtk2_ardour/patch_change.cc +++ b/gtk2_ardour/patch_change.cc @@ -199,12 +199,12 @@ PatchChange::event_handler (GdkEvent* ev) case GDK_Up: case GDK_KP_Up: case GDK_uparrow: - _region.previous_patch (*this); + _region.step_patch (*this, 0, -1); break; case GDK_Down: case GDK_KP_Down: case GDK_downarrow: - _region.next_patch (*this); + _region.step_patch (*this, 0, 1); break; default: break; @@ -213,10 +213,10 @@ PatchChange::event_handler (GdkEvent* ev) case GDK_SCROLL: if (ev->scroll.direction == GDK_SCROLL_UP) { - _region.previous_patch (*this); + _region.step_patch (*this, 0, -1); return true; } else if (ev->scroll.direction == GDK_SCROLL_DOWN) { - _region.next_patch (*this); + _region.step_patch (*this, 0, 1); return true; } break; diff --git a/gtk2_ardour/patch_change_dialog.cc b/gtk2_ardour/patch_change_dialog.cc index 23941e1545..9072801c83 100644 --- a/gtk2_ardour/patch_change_dialog.cc +++ b/gtk2_ardour/patch_change_dialog.cc @@ -285,7 +285,7 @@ PatchChangeDialog::set_active_patch_combo () boost::replace_all (n, "_", " "); MIDI::Name::PatchPrimaryKey const & key = (*j)->patch_primary_key (); - if (key.program_number == _program.get_value() - 1) { + if (key.program() == _program.get_value() - 1) { _ignore_signals = true; _patch_combo.set_active_text (n); _ignore_signals = false; diff --git a/gtk2_ardour/region_view.cc b/gtk2_ardour/region_view.cc index 678ac4dbaf..bd80e27acb 100644 --- a/gtk2_ardour/region_view.cc +++ b/gtk2_ardour/region_view.cc @@ -18,7 +18,6 @@ */ #include -#include #include #include -- cgit v1.2.3