X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Fredirect_box.cc;h=fcd04c9bd7c51dd4bc7e676af9d5d36f56c12612;hb=2592a320d42dd4a157ee16101c042d875d3142be;hp=15442bbaeb2fa5ba8a8493aa48973dc64408cc5c;hpb=202fa5bb783a5d70c49aba8e3bddef96776ba703;p=ardour.git diff --git a/gtk2_ardour/redirect_box.cc b/gtk2_ardour/redirect_box.cc index 15442bbaeb..fcd04c9bd7 100644 --- a/gtk2_ardour/redirect_box.cc +++ b/gtk2_ardour/redirect_box.cc @@ -19,10 +19,11 @@ */ #include -#include #include +#include + #include #include @@ -37,7 +38,7 @@ #include #include #include -#include +#include #include #include #include @@ -53,7 +54,6 @@ #include "route_redirect_selection.h" #include "mixer_ui.h" #include "actions.h" - #include "plugin_ui.h" #include "send_ui.h" #include "io_selector.h" @@ -62,8 +62,13 @@ #include "i18n.h" +#ifdef HAVE_AUDIOUNIT +#include "au_pluginui.h" +#endif + using namespace sigc; using namespace ARDOUR; +using namespace PBD; using namespace Gtk; using namespace Glib; using namespace Gtkmm2ext; @@ -74,7 +79,7 @@ bool RedirectBox::get_colors = true; Gdk::Color* RedirectBox::active_redirect_color; Gdk::Color* RedirectBox::inactive_redirect_color; -RedirectBox::RedirectBox (Placement pcmnt, Session& sess, Route& rt, PluginSelector &plugsel, +RedirectBox::RedirectBox (Placement pcmnt, Session& sess, boost::shared_ptr rt, PluginSelector &plugsel, RouteRedirectSelection & rsel, bool owner_is_mixer) : _route(rt), _session(sess), @@ -130,7 +135,7 @@ RedirectBox::RedirectBox (Placement pcmnt, Session& sess, Route& rt, PluginSelec pack_start (redirect_eventbox, true, true); - _route.redirects_changed.connect (mem_fun(*this, &RedirectBox::redisplay_redirects)); + _route->redirects_changed.connect (mem_fun(*this, &RedirectBox::redisplay_redirects)); redirect_eventbox.signal_enter_notify_event().connect (bind (sigc::ptr_fun (RedirectBox::enter_box), this)); @@ -151,18 +156,18 @@ RedirectBox::~RedirectBox () } void -RedirectBox::object_drop (string type, uint32_t cnt, void** ptr) +RedirectBox::object_drop (string type, uint32_t cnt, const boost::shared_ptr* ptr) { - if (type != "redirects" || cnt == 0 || ptr == 0) { + if (type != "redirects" || cnt == 0 || !ptr) { return; } /* do something with the dropped redirects */ - list redirects; - + list > redirects; + for (uint32_t n = 0; n < cnt; ++n) { - redirects.push_back ((Redirect*) ptr[n]); + redirects.push_back (ptr[n]); } paste_redirect_list (redirects); @@ -187,21 +192,21 @@ RedirectBox::set_width (Width w) } void -RedirectBox::remove_redirect_gui (Redirect *redirect) +RedirectBox::remove_redirect_gui (boost::shared_ptr redirect) { - Insert *insert = 0; - Send *send = 0; - PortInsert *port_insert = 0; + boost::shared_ptr insert; + boost::shared_ptr send; + boost::shared_ptr port_insert; - if ((insert = dynamic_cast (redirect)) != 0) { + if ((insert = boost::dynamic_pointer_cast (redirect)) != 0) { - if ((port_insert = dynamic_cast (insert)) != 0) { + if ((port_insert = boost::dynamic_pointer_cast (insert)) != 0) { PortInsertUI *io_selector = reinterpret_cast (port_insert->get_gui()); port_insert->set_gui (0); delete io_selector; } - } else if ((send = dynamic_cast (insert)) != 0) { + } else if ((send = boost::dynamic_pointer_cast (insert)) != 0) { SendUIWindow *sui = reinterpret_cast (send->get_gui()); send->set_gui (0); delete sui; @@ -243,7 +248,7 @@ RedirectBox::show_redirect_menu (gint arg) paste_action->set_sensitive (!_rr_selection.redirects.empty()); - redirect_menu->popup (1, 0); + redirect_menu->popup (1, arg); } void @@ -266,7 +271,7 @@ RedirectBox::redirect_button_press_event (GdkEventButton *ev) TreeViewColumn* column; int cellx; int celly; - Redirect* redirect = 0; + boost::shared_ptr redirect; int ret = false; bool selected = false; @@ -293,7 +298,7 @@ RedirectBox::redirect_button_press_event (GdkEventButton *ev) } else if (Keyboard::is_context_menu_event (ev)) { - show_redirect_menu(0); + show_redirect_menu(ev->time); ret = true; } else if (redirect && ev->button == 2 && ev->state == 0) { @@ -302,7 +307,12 @@ RedirectBox::redirect_button_press_event (GdkEventButton *ev) ret = true; } + else if (redirect && ev->button == 1 && selected) { + // this is purely informational but necessary + RedirectSelected (redirect); // emit + } + return ret; } @@ -346,32 +356,32 @@ RedirectBox::choose_plugin () } void -RedirectBox::insert_plugin_chosen (Plugin *plugin) +RedirectBox::insert_plugin_chosen (boost::shared_ptr plugin) { if (plugin) { - Redirect *redirect = new PluginInsert (_session, *plugin, _placement); + boost::shared_ptr redirect (new PluginInsert (_session, plugin, _placement)); redirect->active_changed.connect (mem_fun(*this, &RedirectBox::show_redirect_active)); uint32_t err_streams; - if (_route.add_redirect (redirect, this, &err_streams)) { + if (_route->add_redirect (redirect, this, &err_streams)) { wierd_plugin_dialog (*plugin, err_streams, _route); - delete redirect; + // XXX SHAREDPTR delete plugin here .. do we even need to care? } } } void -RedirectBox::wierd_plugin_dialog (Plugin& p, uint32_t streams, IO& io) +RedirectBox::wierd_plugin_dialog (Plugin& p, uint32_t streams, boost::shared_ptr io) { ArdourDialog dialog ("wierd plugin dialog"); Label label; /* i hate this kind of code */ - if (streams > p.get_info().n_inputs) { + if (streams > p.get_info()->n_inputs) { label.set_text (string_compose (_( "You attempted to add a plugin (%1).\n" "The plugin has %2 inputs\n" @@ -381,9 +391,9 @@ RedirectBox::wierd_plugin_dialog (Plugin& p, uint32_t streams, IO& io) "This makes no sense - you are throwing away\n" "part of the signal."), p.name(), - p.get_info().n_inputs, + p.get_info()->n_inputs, streams)); - } else if (streams < p.get_info().n_inputs) { + } else if (streams < p.get_info()->n_inputs) { label.set_text (string_compose (_( "You attempted to add a plugin (%1).\n" "The plugin has %2 inputs\n" @@ -394,7 +404,7 @@ RedirectBox::wierd_plugin_dialog (Plugin& p, uint32_t streams, IO& io) "side-chain inputs. A future version of Ardour will\n" "support this type of configuration."), p.name(), - p.get_info().n_inputs, + p.get_info()->n_inputs, streams)); } else { label.set_text (string_compose (_( @@ -408,10 +418,10 @@ RedirectBox::wierd_plugin_dialog (Plugin& p, uint32_t streams, IO& io) "\n" "Ardour does not understand what to do in such situations.\n"), p.name(), - p.get_info().n_inputs, - p.get_info().n_outputs, - io.n_inputs(), - io.n_outputs(), + p.get_info()->n_inputs, + p.get_info()->n_outputs, + io->n_inputs(), + io->n_outputs(), streams)); } @@ -423,46 +433,42 @@ RedirectBox::wierd_plugin_dialog (Plugin& p, uint32_t streams, IO& io) dialog.set_modal (true); dialog.show_all (); - // GTK2FIX - //dialog.realize(); - //dialog.get_window()->set_decorations (Gdk::WMDecoration (GDK_DECOR_BORDER|GDK_DECOR_RESIZEH)); - dialog.run (); } void RedirectBox::choose_insert () { - Redirect *redirect = new PortInsert (_session, _placement); + boost::shared_ptr redirect (new PortInsert (_session, _placement)); redirect->active_changed.connect (mem_fun(*this, &RedirectBox::show_redirect_active)); - _route.add_redirect (redirect, this); + _route->add_redirect (redirect, this); } void RedirectBox::choose_send () { - Send *send = new Send (_session, _placement); + boost::shared_ptr send (new Send (_session, _placement)); /* XXX need redirect lock on route */ - send->ensure_io (0, _route.max_redirect_outs(), false, this); + send->ensure_io (0, _route->max_redirect_outs(), false, this); - IOSelectorWindow *ios = new IOSelectorWindow (_session, *send, false, true); + IOSelectorWindow *ios = new IOSelectorWindow (_session, send, false, true); ios->show_all (); - ios->selector().Finished.connect (bind (mem_fun(*this, &RedirectBox::send_io_finished), static_cast(send), ios)); + ios->selector().Finished.connect (bind (mem_fun(*this, &RedirectBox::send_io_finished), boost::static_pointer_cast(send), ios)); } void -RedirectBox::send_io_finished (IOSelector::Result r, Redirect* redirect, IOSelectorWindow* ios) +RedirectBox::send_io_finished (IOSelector::Result r, boost::shared_ptr redirect, IOSelectorWindow* ios) { switch (r) { case IOSelector::Cancelled: - delete redirect; + // delete redirect; XXX SHAREDPTR HOW TO DESTROY THE REDIRECT ? do we even need to think about it? break; case IOSelector::Accepted: - _route.add_redirect (redirect, this); + _route->add_redirect (redirect, this); break; } @@ -485,7 +491,8 @@ RedirectBox::redisplay_redirects (void *src) redirect_active_connections.clear (); redirect_name_connections.clear (); - _route.foreach_redirect (this, &RedirectBox::add_redirect_to_display); + void (RedirectBox::*pmf)(boost::shared_ptr) = &RedirectBox::add_redirect_to_display; + _route->foreach_redirect (this, pmf); switch (_placement) { case PreFader: @@ -498,33 +505,33 @@ RedirectBox::redisplay_redirects (void *src) } void -RedirectBox::add_redirect_to_display (Redirect *redirect) +RedirectBox::add_redirect_to_display (boost::shared_ptr redirect) { if (redirect->placement() != _placement) { return; } Gtk::TreeModel::Row row = *(model->append()); - row[columns.text] = redirect_name (*redirect); + row[columns.text] = redirect_name (redirect); row[columns.redirect] = redirect; - show_redirect_active (redirect, this); + show_redirect_active (redirect.get(), this); redirect_active_connections.push_back (redirect->active_changed.connect (mem_fun(*this, &RedirectBox::show_redirect_active))); redirect_name_connections.push_back (redirect->name_changed.connect (bind (mem_fun(*this, &RedirectBox::show_redirect_name), redirect))); } string -RedirectBox::redirect_name (Redirect& redirect) +RedirectBox::redirect_name (boost::shared_ptr redirect) { - Send *send; + boost::shared_ptr send; string name_display; - if (!redirect.active()) { + if (!redirect->active()) { name_display = " ("; } - if ((send = dynamic_cast (&redirect)) != 0) { + if ((send = boost::dynamic_pointer_cast (redirect)) != 0) { name_display += '>'; @@ -539,7 +546,7 @@ RedirectBox::redirect_name (Redirect& redirect) name_display += send->name().substr (lbracket+1, lbracket-rbracket-1); break; case Narrow: - name_display += short_version (send->name().substr (lbracket+1, lbracket-rbracket-1), 4); + name_display += PBD::short_version (send->name().substr (lbracket+1, lbracket-rbracket-1), 4); break; } @@ -547,16 +554,16 @@ RedirectBox::redirect_name (Redirect& redirect) switch (_width) { case Wide: - name_display += redirect.name(); + name_display += redirect->name(); break; case Narrow: - name_display += short_version (redirect.name(), 5); + name_display += PBD::short_version (redirect->name(), 5); break; } } - if (!redirect.active()) { + if (!redirect->active()) { name_display += ')'; } @@ -578,34 +585,36 @@ RedirectBox::build_redirect_tooltip (EventBox& box, string start) } void -RedirectBox::show_redirect_name (void* src, Redirect *redirect) +RedirectBox::show_redirect_name (void* src, boost::shared_ptr redirect) { ENSURE_GUI_THREAD(bind (mem_fun(*this, &RedirectBox::show_redirect_name), src, redirect)); - - show_redirect_active (redirect, src); + show_redirect_active (redirect.get(), src); } void -RedirectBox::show_redirect_active (Redirect *redirect, void *src) +RedirectBox::show_redirect_active (Redirect* redirect, void *src) { ENSURE_GUI_THREAD(bind (mem_fun(*this, &RedirectBox::show_redirect_active), redirect, src)); Gtk::TreeModel::Children children = model->children(); Gtk::TreeModel::Children::iterator iter = children.begin(); - while( iter != children.end()) - { - if ((*iter)[columns.redirect] == redirect) - break; - iter++; - } + while (iter != children.end()) { - (*iter)[columns.text] = redirect_name (*redirect); + boost::shared_ptr r = (*iter)[columns.redirect]; - if (redirect->active()) { - (*iter)[columns.color] = *active_redirect_color; - } else { - (*iter)[columns.color] = *inactive_redirect_color; + if (r.get() == redirect) { + (*iter)[columns.text] = redirect_name (r); + + if (redirect->active()) { + (*iter)[columns.color] = *active_redirect_color; + } else { + (*iter)[columns.color] = *inactive_redirect_color; + } + break; + } + + iter++; } } @@ -624,12 +633,12 @@ RedirectBox::compute_redirect_sort_keys () Gtk::TreeModel::Children children = model->children(); for (Gtk::TreeModel::Children::iterator iter = children.begin(); iter != children.end(); ++iter) { - Redirect *redirect = (*iter)[columns.redirect]; - redirect->set_sort_key (sort_key); + boost::shared_ptr r = (*iter)[columns.redirect]; + r->set_sort_key (sort_key); sort_key++; } - if (_route.sort_redirects ()) { + if (_route->sort_redirects ()) { redisplay_redirects (0); @@ -658,7 +667,7 @@ outputs do not work correctly.")); void RedirectBox::rename_redirects () { - vector to_be_renamed; + vector > to_be_renamed; get_selected_redirects (to_be_renamed); @@ -666,7 +675,7 @@ RedirectBox::rename_redirects () return; } - for (vector::iterator i = to_be_renamed.begin(); i != to_be_renamed.end(); ++i) { + for (vector >::iterator i = to_be_renamed.begin(); i != to_be_renamed.end(); ++i) { rename_redirect (*i); } } @@ -674,7 +683,7 @@ RedirectBox::rename_redirects () void RedirectBox::cut_redirects () { - vector to_be_removed; + vector > to_be_removed; get_selected_redirects (to_be_removed); @@ -689,7 +698,7 @@ RedirectBox::cut_redirects () _rr_selection.set (to_be_removed); - for (vector::iterator i = to_be_removed.begin(); i != to_be_removed.end(); ++i) { + for (vector >::iterator i = to_be_removed.begin(); i != to_be_removed.end(); ++i) { void* gui = (*i)->get_gui (); @@ -697,7 +706,7 @@ RedirectBox::cut_redirects () static_cast(gui)->hide (); } - if (_route.remove_redirect (*i, this)) { + if (_route->remove_redirect (*i, this)) { /* removal failed */ _rr_selection.remove (*i); } @@ -708,8 +717,8 @@ RedirectBox::cut_redirects () void RedirectBox::copy_redirects () { - vector to_be_copied; - vector copies; + vector > to_be_copied; + vector > copies; get_selected_redirects (to_be_copied); @@ -717,34 +726,31 @@ RedirectBox::copy_redirects () return; } - for (vector::iterator i = to_be_copied.begin(); i != to_be_copied.end(); ++i) { - copies.push_back (Redirect::clone (**i)); + for (vector >::iterator i = to_be_copied.begin(); i != to_be_copied.end(); ++i) { + copies.push_back (Redirect::clone (*i)); } _rr_selection.set (copies); } gint -RedirectBox::idle_delete_redirect (Redirect *redirect) +RedirectBox::idle_delete_redirect (boost::shared_ptr redirect) { /* NOT copied to _mixer.selection() */ - if (_route.remove_redirect (redirect, this)) { - /* removal failed */ - return FALSE; - } - - delete redirect; + _route->remove_redirect (redirect, this); return FALSE; } void -RedirectBox::rename_redirect (Redirect* redirect) +RedirectBox::rename_redirect (boost::shared_ptr redirect) { ArdourPrompter name_prompter (true); string result; name_prompter.set_prompt (_("rename redirect")); name_prompter.set_initial_text (redirect->name()); + name_prompter.add_button (_("Rename"), Gtk::RESPONSE_ACCEPT); + name_prompter.set_response_sensitive (Gtk::RESPONSE_ACCEPT, false); name_prompter.show_all (); switch (name_prompter.run ()) { @@ -762,7 +768,7 @@ RedirectBox::rename_redirect (Redirect* redirect) } void -RedirectBox::cut_redirect (Redirect *redirect) +RedirectBox::cut_redirect (boost::shared_ptr redirect) { /* this essentially transfers ownership of the redirect of the redirect from the route to the mixer @@ -777,15 +783,15 @@ RedirectBox::cut_redirect (Redirect *redirect) static_cast(gui)->hide (); } - if (_route.remove_redirect (redirect, this)) { + if (_route->remove_redirect (redirect, this)) { _rr_selection.remove (redirect); } } void -RedirectBox::copy_redirect (Redirect *redirect) +RedirectBox::copy_redirect (boost::shared_ptr redirect) { - Redirect* copy = Redirect::clone (*redirect); + boost::shared_ptr copy = Redirect::clone (redirect); _rr_selection.add (copy); } @@ -800,22 +806,19 @@ RedirectBox::paste_redirects () } void -RedirectBox::paste_redirect_list (list& redirects) +RedirectBox::paste_redirect_list (list >& redirects) { - list copies; + list > copies; - for (list::iterator i = redirects.begin(); i != redirects.end(); ++i) { + for (list >::iterator i = redirects.begin(); i != redirects.end(); ++i) { - Redirect* copy = Redirect::clone (**i); + boost::shared_ptr copy = Redirect::clone (*i); copy->set_placement (_placement, this); copies.push_back (copy); } - if (_route.add_redirects (copies, this)) { - for (list::iterator i = copies.begin(); i != copies.end(); ++i) { - delete *i; - } + if (_route->add_redirects (copies, this)) { string msg = _( "Copying the set of redirects on the clipboard failed,\n\ @@ -827,19 +830,19 @@ could not match the configuration of this track."); } void -RedirectBox::activate_redirect (Redirect *r) +RedirectBox::activate_redirect (boost::shared_ptr r) { r->set_active (true, 0); } void -RedirectBox::deactivate_redirect (Redirect *r) +RedirectBox::deactivate_redirect (boost::shared_ptr r) { r->set_active (false, 0); } void -RedirectBox::get_selected_redirects (vector& redirects) +RedirectBox::get_selected_redirects (vector >& redirects) { vector pathlist = redirect_display.get_selection()->get_selected_rows(); @@ -848,12 +851,12 @@ RedirectBox::get_selected_redirects (vector& redirects) } void -RedirectBox::for_selected_redirects (void (RedirectBox::*pmf)(Redirect*)) +RedirectBox::for_selected_redirects (void (RedirectBox::*pmf)(boost::shared_ptr)) { vector pathlist = redirect_display.get_selection()->get_selected_rows(); for (vector::iterator iter = pathlist.begin(); iter != pathlist.end(); ++iter) { - Redirect* redirect = (*(model->get_iter(*iter)))[columns.redirect]; + boost::shared_ptr redirect = (*(model->get_iter(*iter)))[columns.redirect]; (this->*pmf)(redirect); } } @@ -864,7 +867,7 @@ RedirectBox::clone_redirects () RouteSelection& routes (_rr_selection.routes); if (!routes.empty()) { - if (_route.copy_redirects (*routes.front(), _placement)) { + if (_route->copy_redirects (*routes.front(), _placement)) { string msg = _( "Copying the set of redirects on the clipboard failed,\n\ probably because the I/O configuration of the plugins\n\ @@ -878,7 +881,7 @@ could not match the configuration of this track."); void RedirectBox::all_redirects_active (bool state) { - _route.all_redirects_active (state); + _route->all_redirects_active (state); } void @@ -887,7 +890,7 @@ RedirectBox::clear_redirects() string prompt; vector choices; - if (dynamic_cast(&_route) != 0) { + if (boost::dynamic_pointer_cast(_route) != 0) { prompt = _("Do you really want to remove all redirects from this track?\n" "(this cannot be undone)"); } else { @@ -895,35 +898,29 @@ RedirectBox::clear_redirects() "(this cannot be undone)"); } - choices.push_back (_("Yes, remove them all")); choices.push_back (_("Cancel")); + choices.push_back (_("Yes, remove them all")); Gtkmm2ext::Choice prompter (prompt, choices); - prompter.chosen.connect(sigc::ptr_fun(Gtk::Main::quit)); - prompter.show_all (); - - Gtk::Main::run (); - - if (prompter.get_choice() == 0) { - _route.clear_redirects (this); + if (prompter.run () == 1) { + _route->clear_redirects (this); } } - void -RedirectBox::edit_redirect (Redirect* redirect) +RedirectBox::edit_redirect (boost::shared_ptr redirect) { - Insert *insert; + boost::shared_ptr insert; - if (dynamic_cast(&_route) != 0) { + if (boost::dynamic_pointer_cast(_route) != 0) { - if (dynamic_cast (&_route)->freeze_state() == AudioTrack::Frozen) { + if (boost::dynamic_pointer_cast (_route)->freeze_state() == AudioTrack::Frozen) { return; } } - if ((insert = dynamic_cast (redirect)) == 0) { + if ((insert = boost::dynamic_pointer_cast (redirect)) == 0) { /* its a send */ @@ -931,7 +928,7 @@ RedirectBox::edit_redirect (Redirect* redirect) return; } - Send *send = dynamic_cast (redirect); + boost::shared_ptr send = boost::dynamic_pointer_cast (redirect); SendUIWindow *send_ui; @@ -940,7 +937,7 @@ RedirectBox::edit_redirect (Redirect* redirect) string title; title = string_compose(_("ardour: %1"), send->name()); - send_ui = new SendUIWindow (*send, _session); + send_ui = new SendUIWindow (send, _session); send_ui->set_title (title); send->set_gui (send_ui); @@ -958,50 +955,67 @@ RedirectBox::edit_redirect (Redirect* redirect) /* its an insert */ - PluginInsert *plugin_insert; - PortInsert *port_insert; + boost::shared_ptr plugin_insert; + boost::shared_ptr port_insert; - if ((plugin_insert = dynamic_cast (insert)) != 0) { + if ((plugin_insert = boost::dynamic_pointer_cast (insert)) != 0) { - PluginUIWindow *plugin_ui; + ARDOUR::PluginType type = plugin_insert->type(); + + if (type == ARDOUR::LADSPA || type == ARDOUR::VST) { + PluginUIWindow *plugin_ui; - if (plugin_insert->get_gui() == 0) { + if (plugin_insert->get_gui() == 0) { - string title; - string maker = plugin_insert->plugin().maker(); - string::size_type email_pos; + string title; + string maker = plugin_insert->plugin()->maker(); + string::size_type email_pos; - if ((email_pos = maker.find_first_of ('<')) != string::npos) { - maker = maker.substr (0, email_pos - 1); - } + if ((email_pos = maker.find_first_of ('<')) != string::npos) { + maker = maker.substr (0, email_pos - 1); + } - if (maker.length() > 32) { - maker = maker.substr (0, 32); - maker += " ..."; - } + if (maker.length() > 32) { + maker = maker.substr (0, 32); + maker += " ..."; + } - title = string_compose(_("ardour: %1: %2 (by %3)"), _route.name(), plugin_insert->name(), maker); + title = string_compose(_("ardour: %1: %2 (by %3)"), _route->name(), plugin_insert->name(), maker); + + plugin_ui = new PluginUIWindow (plugin_insert); + if (_owner_is_mixer) { + ARDOUR_UI::instance()->the_mixer()->ensure_float (*plugin_ui); + } else { + ARDOUR_UI::instance()->the_editor().ensure_float (*plugin_ui); + } + plugin_ui->set_title (title); + plugin_insert->set_gui (plugin_ui); - plugin_ui = new PluginUIWindow (_session.engine(), *plugin_insert); - if (_owner_is_mixer) { - ARDOUR_UI::instance()->the_mixer()->ensure_float (*plugin_ui); } else { - ARDOUR_UI::instance()->the_editor().ensure_float (*plugin_ui); + plugin_ui = reinterpret_cast (plugin_insert->get_gui()); } - plugin_ui->set_title (title); - plugin_insert->set_gui (plugin_ui); - - } else { - plugin_ui = reinterpret_cast (plugin_insert->get_gui()); - } - if (plugin_ui->is_visible()) { - plugin_ui->get_window()->raise (); + if (plugin_ui->is_visible()) { + plugin_ui->get_window()->raise (); + } else { + plugin_ui->show_all (); + } +#ifdef HAVE_AUDIOUNIT + } else if (type == ARDOUR::AudioUnit) { + AUPluginUI* plugin_ui; + if (plugin_insert->get_gui() == 0) { + plugin_ui = new AUPluginUI (plugin_insert); + } else { + plugin_ui = reinterpret_cast (plugin_insert->get_gui()); + } + + // raise window, somehow +#endif } else { - plugin_ui->show_all (); + warning << "Unsupported plugin sent to RedirectBox::edit_redirect()" << endmsg; + return; } - - } else if ((port_insert = dynamic_cast (insert)) != 0) { + } else if ((port_insert = boost::dynamic_pointer_cast (insert)) != 0) { if (!_session.engine().connected()) { MessageDialog msg ( _("Not connected to JACK - no I/O changes are possible")); @@ -1012,7 +1026,7 @@ RedirectBox::edit_redirect (Redirect* redirect) PortInsertWindow *io_selector; if (port_insert->get_gui() == 0) { - io_selector = new PortInsertWindow (_session, *port_insert); + io_selector = new PortInsertWindow (_session, port_insert); port_insert->set_gui (io_selector); } else {