X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Fport_matrix.cc;h=1b588511fdc47c7555555b634b9d70befc4544cc;hb=cf52d6e4b40111eb04b244ec054055a4ec15dbe0;hp=6bd9d962629d499c007760ac22fc584c6232062c;hpb=86343b6c15a3a43f082802484d2bc813d34db821;p=ardour.git diff --git a/gtk2_ardour/port_matrix.cc b/gtk2_ardour/port_matrix.cc index 6bd9d96262..1b588511fd 100644 --- a/gtk2_ardour/port_matrix.cc +++ b/gtk2_ardour/port_matrix.cc @@ -32,17 +32,19 @@ #include "ardour/session.h" #include "ardour/route.h" #include "ardour/audioengine.h" +#include "gtkmm2ext/utils.h" #include "port_matrix.h" #include "port_matrix_body.h" #include "port_matrix_component.h" #include "ardour_dialog.h" -#include "i18n.h" +#include "pbd/i18n.h" #include "gui_thread.h" #include "utils.h" using namespace std; using namespace Gtk; using namespace ARDOUR; +using namespace ARDOUR_UI_UTILS; /** PortMatrix constructor. * @param session Our session. @@ -151,14 +153,13 @@ PortMatrix::init () _session->RouteAdded.connect (_session_connections, invalidator (*this), boost::bind (&PortMatrix::routes_changed, this), gui_context()); /* and also bundles */ - _session->BundleAdded.connect (_session_connections, invalidator (*this), boost::bind (&PortMatrix::setup_global_ports, this), gui_context()); - _session->BundleRemoved.connect (_session_connections, invalidator (*this), boost::bind (&PortMatrix::setup_global_ports, this), gui_context()); + _session->BundleAddedOrRemoved.connect (_session_connections, invalidator (*this), boost::bind (&PortMatrix::setup_global_ports, this), gui_context()); /* and also ports */ _session->engine().PortRegisteredOrUnregistered.connect (_session_connections, invalidator (*this), boost::bind (&PortMatrix::setup_global_ports, this), gui_context()); /* watch for route order keys changing, which changes the order of things in our global ports list(s) */ - Route::SyncOrderKeys.connect (_session_connections, invalidator (*this), boost::bind (&PortMatrix::setup_global_ports_proxy, this), gui_context()); + PresentationInfo::Change.connect (_session_connections, invalidator (*this), boost::bind (&PortMatrix::setup_global_ports_proxy, this), gui_context()); /* Part 3: other stuff */ @@ -248,17 +249,26 @@ PortMatrix::setup_scrollbars () { Adjustment* a = _hscroll.get_adjustment (); a->set_lower (0); - a->set_upper (_body->full_scroll_width()); a->set_page_size (_body->alloc_scroll_width()); a->set_step_increment (32); a->set_page_increment (128); + /* Set the adjustment to zero if the size has changed.*/ + if (a->get_upper() != _body->full_scroll_width()) { + a->set_upper (_body->full_scroll_width()); + a->set_value (0); + } + a = _vscroll.get_adjustment (); a->set_lower (0); - a->set_upper (_body->full_scroll_height()); a->set_page_size (_body->alloc_scroll_height()); a->set_step_increment (32); a->set_page_increment (128); + + if (a->get_upper() != _body->full_scroll_height()) { + a->set_upper (_body->full_scroll_height()); + a->set_value (0); + } } /** Disassociate all of our ports from each other */ @@ -441,7 +451,7 @@ PortMatrix::popup_menu (BundleChannel column, BundleChannel row, uint32_t t) sub.push_back (MenuElem (buf, sigc::bind (sigc::mem_fun (*this, &PortMatrix::add_channel_proxy), w, *i))); } } - + /* Now add other ones */ for (DataType::iterator i = DataType::begin(); i != DataType::end(); ++i) { if (!should_show (*i)) { @@ -488,12 +498,12 @@ PortMatrix::popup_menu (BundleChannel column, BundleChannel row, uint32_t t) /* we're looking just at bundles, or our bundle has only one channel, so just offer to disassociate all on the bundle. */ - + snprintf (buf, sizeof (buf), _("%s all"), disassociation_verb().c_str()); sub.push_back ( MenuElem (buf, sigc::bind (sigc::mem_fun (*this, &PortMatrix::disassociate_all_on_bundle), w, dim)) ); - + } else if (c != 0) { if (bc[dim].channel != -1) { @@ -534,7 +544,7 @@ PortMatrix::popup_menu (BundleChannel column, BundleChannel row, uint32_t t) items.push_back (MenuElem (_("Flip"), sigc::mem_fun (*this, &PortMatrix::flip))); items.back().set_sensitive (can_flip ()); - + _menu->popup (1, t); } @@ -609,6 +619,7 @@ PortMatrix::disassociate_all_on_channel (boost::weak_ptr bundle, uint32_ void PortMatrix::setup_global_ports () { + if (!_session || _session->deletion_in_progress()) return; ENSURE_GUI_THREAD (*this, &PortMatrix::setup_global_ports) for (int i = 0; i < 2; ++i) { @@ -624,7 +635,7 @@ PortMatrix::setup_global_ports_proxy () /* Avoid a deadlock by calling this in an idle handler: see IOSelector::io_changed_proxy for a discussion. */ - + Glib::signal_idle().connect_once (sigc::mem_fun (*this, &PortMatrix::setup_global_ports)); } @@ -707,7 +718,7 @@ PortMatrix::io_from_bundle (boost::shared_ptr b) const bool PortMatrix::can_add_channels (boost::shared_ptr b) const { - return io_from_bundle (b); + return io_from_bundle (b) != 0; } void @@ -718,9 +729,7 @@ PortMatrix::add_channel (boost::shared_ptr b, DataType t) if (io) { int const r = io->add_port ("", this, t); if (r == -1) { - Gtk::MessageDialog msg (_("It is not possible to add a port here, as the first processor in the track or buss cannot " - "support the new configuration." - )); + Gtk::MessageDialog msg (_("It is not possible to add a port here.")); msg.set_title (_("Cannot add port")); msg.run (); } @@ -730,29 +739,37 @@ PortMatrix::add_channel (boost::shared_ptr b, DataType t) bool PortMatrix::can_remove_channels (boost::shared_ptr b) const { - return io_from_bundle (b); + return io_from_bundle (b) != 0; } void PortMatrix::remove_channel (ARDOUR::BundleChannel b) { + std::string errmsg; boost::shared_ptr io = io_from_bundle (b.bundle); + boost::shared_ptr p = io->nth (b.channel); - if (io) { - boost::shared_ptr p = io->nth (b.channel); - if (p) { - int const r = io->remove_port (p, this); - if (r == -1) { - ArdourDialog d (_("Port removal not allowed")); - Label l (_("This port cannot be removed.\nEither the first plugin in the track or buss cannot accept\nthe new number of inputs or the last plugin has more outputs.")); - d.get_vbox()->pack_start (l); - d.add_button (Stock::OK, RESPONSE_ACCEPT); - d.set_modal (true); - d.show_all (); - d.run (); - } + if (!io || !p) { + return; + } + + if (io->n_ports ().n_total () == 1) { + errmsg = _("The last port cannot be removed"); + } else { + if (-1 == io->remove_port (p, this)) { + errmsg = _("This port cannot be removed."); } } + + if (!errmsg.empty ()) { + ArdourDialog d (_("Port removal not allowed")); + Label l (errmsg); + d.get_vbox()->pack_start (l); + d.add_button (Stock::OK, RESPONSE_ACCEPT); + d.set_modal (true); + d.show_all (); + d.run (); + } } void @@ -925,7 +942,7 @@ PortMatrix::visible_ports (int d) const the TOP_TO_RIGHT arrangement we reverse the order of the vertical tabs in setup_notebooks (). */ - + int n = 0; if (d == _row_index) { if (_arrangement == LEFT_TO_BOTTOM) { @@ -997,11 +1014,11 @@ PortMatrix::update_tab_highlighting () if (!_session) { return; } - + for (int i = 0; i < 2; ++i) { Gtk::Notebook* notebook = row_index() == i ? &_vnotebook : &_hnotebook; - + PortGroupList const * gl = ports (i); int p = 0; for (PortGroupList::List::const_iterator j = gl->begin(); j != gl->end(); ++j) { @@ -1031,7 +1048,7 @@ PortMatrix::update_tab_highlighting () label->set_text ((*j)->name); } else if (c.length() && c[0] != '<' && has_connection) { /* this label is not marked up with but should be */ - label->set_markup (string_compose ("%1", Glib::Markup::escape_text ((*j)->name))); + label->set_markup (string_compose ("%1", Gtkmm2ext::markup_escape_text ((*j)->name))); } ++p; @@ -1137,7 +1154,7 @@ PortMatrix::get_association (PortMatrixNode node) const } - /* NOTREACHED */ + abort(); /* NOTREACHED */ return PortMatrixNode::NOT_ASSOCIATED; } @@ -1156,7 +1173,7 @@ pair PortMatrix::check_flip () const { /* Look for the row's port group name in the columns */ - + int new_column = 0; boost::shared_ptr r = visible_ports (_row_index); PortGroupList::List::const_iterator i = _ports[_column_index].begin(); @@ -1170,7 +1187,7 @@ PortMatrix::check_flip () const } /* Look for the column's port group name in the rows */ - + int new_row = 0; boost::shared_ptr c = visible_ports (_column_index); i = _ports[_row_index].begin();