Remove use of invalidated iterator (#4826).
[ardour.git] / gtk2_ardour / ardour_ui.cc
index 1fa4b21b56e89e4b7ff3885467e1f821928d2375..d9142b8b1aa6eb9c182717a8383087a35a4573a6 100644 (file)
@@ -62,6 +62,7 @@
 #include "ardour/ardour.h"
 #include "ardour/callback.h"
 #include "ardour/profile.h"
+#include "ardour/plugin_manager.h"
 #include "ardour/session_directory.h"
 #include "ardour/session_route.h"
 #include "ardour/session_state_utils.h"
@@ -78,6 +79,7 @@
 #include "ardour/midi_track.h"
 #include "ardour/filesystem_paths.h"
 #include "ardour/filename_extensions.h"
+#include "ardour/process_thread.h"
 
 typedef uint64_t microseconds_t;
 
@@ -91,6 +93,7 @@ typedef uint64_t microseconds_t;
 #include "engine_dialog.h"
 #include "gain_meter.h"
 #include "global_port_matrix.h"
+#include "gui_object.h"
 #include "gui_thread.h"
 #include "keyboard.h"
 #include "location_ui.h"
@@ -133,19 +136,13 @@ ARDOUR_UI::ARDOUR_UI (int *argcp, char **argvp[])
 
        : Gtkmm2ext::UI (PROGRAM_NAME, argcp, argvp)
 
-       , primary_clock (new AudioClock (X_("primary"), false, X_("TransportClockDisplay"), true, true, false, true))
-       , secondary_clock (new AudioClock (X_("secondary"), false, X_("SecondaryClockDisplay"), true, true, false, true))
-       , preroll_clock (new AudioClock (X_("preroll"), false, X_("PreRollClock"), true, false, true))
-       , postroll_clock (new AudioClock (X_("postroll"), false, X_("PostRollClock"), true, false, true))
-
-         /* preroll stuff */
-
-       , preroll_button (_("pre\nroll"))
-       , postroll_button (_("post\nroll"))
+       , gui_object_state (new GUIObjectState)
+       , primary_clock (new AudioClock (X_("primary"), false, X_("transport"), true, true, false, true))
+       , secondary_clock (new AudioClock (X_("secondary"), false, X_("secondary"), true, true, false, true))
 
          /* big clock */
 
-       , big_clock (new AudioClock (X_("bigclock"), false, "BigClockNonRecording", true, true, false, false))
+       , big_clock (new AudioClock (X_("bigclock"), false, "big", true, true, false, false))
 
          /* transport */
 
@@ -157,27 +154,19 @@ ARDOUR_UI::ARDOUR_UI (int *argcp, char **argvp[])
        , play_selection_controllable (new TransportControllable ("transport play selection", *this, TransportControllable::PlaySelection))
        , rec_controllable (new TransportControllable ("transport rec-enable", *this, TransportControllable::RecordEnable))
 
-       , roll_button (roll_controllable)
-       , stop_button (stop_controllable)
-       , goto_start_button (goto_start_controllable)
-       , goto_end_button (goto_end_controllable)
-       , auto_loop_button (auto_loop_controllable)
-       , play_selection_button (play_selection_controllable)
-       , rec_button (rec_controllable)
-
-       , punch_in_button (_("Punch In"))
-       , punch_out_button (_("Punch Out"))
-       , auto_return_button (_("Auto Return"))
-       , auto_play_button (_("Auto Play"))
-       , auto_input_button (_("Auto Input"))
-       , click_button (_("Click"))
-       , time_master_button (_("time\nmaster"))
-
-       , auditioning_alert_button (_("AUDITION"))
-       , solo_alert_button (_("SOLO"))
+       , auto_return_button (ArdourButton::led_default_elements)
+       , auto_play_button (ArdourButton::led_default_elements)
+       , auto_input_button (ArdourButton::led_default_elements)
+
+       , auditioning_alert_button (_("audition"))
+       , solo_alert_button (_("solo"))
+       , feedback_alert_button (_("feedback"))
 
        , error_log_button (_("Errors"))
 
+       , _status_bar_visibility (X_("status-bar"))
+       , _feedback_exists (false)
+
 {
        using namespace Gtk::Menu_Helpers;
 
@@ -216,7 +205,6 @@ ARDOUR_UI::ARDOUR_UI (int *argcp, char **argvp[])
        big_clock_resize_in_progress = false;
        session_selector_window = 0;
        last_key_press_time = 0;
-       _will_create_new_session_automatically = false;
        add_route_dialog = 0;
        route_params = 0;
        bundle_manager = 0;
@@ -232,14 +220,30 @@ ARDOUR_UI::ARDOUR_UI (int *argcp, char **argvp[])
        original_big_clock_height = -1;
        original_big_clock_font_size = 0;
 
-       roll_button.unset_flags (Gtk::CAN_FOCUS);
-       stop_button.unset_flags (Gtk::CAN_FOCUS);
-       goto_start_button.unset_flags (Gtk::CAN_FOCUS);
-       goto_end_button.unset_flags (Gtk::CAN_FOCUS);
-       auto_loop_button.unset_flags (Gtk::CAN_FOCUS);
-       play_selection_button.unset_flags (Gtk::CAN_FOCUS);
-       rec_button.unset_flags (Gtk::CAN_FOCUS);
-       join_play_range_button.unset_flags (Gtk::CAN_FOCUS);
+       roll_button.set_elements (ArdourButton::Element (ArdourButton::Body|ArdourButton::Text));
+       play_selection_button.set_elements (ArdourButton::Element (ArdourButton::Body|ArdourButton::Text));
+       
+       roll_button.set_controllable (roll_controllable);
+       stop_button.set_controllable (stop_controllable);
+       goto_start_button.set_controllable (goto_start_controllable);
+       goto_end_button.set_controllable (goto_end_controllable);
+       auto_loop_button.set_controllable (auto_loop_controllable);
+       play_selection_button.set_controllable (play_selection_controllable);
+       rec_button.set_controllable (rec_controllable);
+
+       roll_button.set_name ("transport button");
+       stop_button.set_name ("transport button");
+       goto_start_button.set_name ("transport button");
+       goto_end_button.set_name ("transport button");
+       auto_loop_button.set_name ("transport button");
+       play_selection_button.set_name ("transport button");
+       rec_button.set_name ("transport recenable button");
+       midi_panic_button.set_name ("transport button");
+
+       goto_start_button.set_tweaks (ArdourButton::ShowClick);
+       goto_end_button.set_tweaks (ArdourButton::ShowClick);
+       midi_panic_button.set_tweaks (ArdourButton::ShowClick);
+       
        last_configure_time= 0;
        last_peak_grab = 0;
 
@@ -262,6 +266,11 @@ ARDOUR_UI::ARDOUR_UI (int *argcp, char **argvp[])
 
        ARDOUR::Session::Quit.connect (forever_connections, MISSING_INVALIDATOR, ui_bind (&ARDOUR_UI::finish, this), gui_context ());
 
+       /* tell the user about feedback */
+
+       ARDOUR::Session::FeedbackDetected.connect (forever_connections, MISSING_INVALIDATOR, ui_bind (&ARDOUR_UI::feedback_detected, this), gui_context ());
+       ARDOUR::Session::SuccessfulGraphSort.connect (forever_connections, MISSING_INVALIDATOR, ui_bind (&ARDOUR_UI::successful_graph_sort, this), gui_context ());
+
        /* handle requests to deal with missing files */
 
        ARDOUR::Session::MissingFile.connect_same_thread (forever_connections, boost::bind (&ARDOUR_UI::missing_file, this, _1, _2, _3));
@@ -282,7 +291,7 @@ ARDOUR_UI::ARDOUR_UI (int *argcp, char **argvp[])
 
                GainMeter::setup_slider_pix ();
                RouteTimeAxisView::setup_slider_pix ();
-               SendProcessorEntry::setup_slider_pix ();
+               ProcessorEntry::setup_slider_pix ();
                SessionEvent::create_per_thread_pool ("GUI", 512);
 
        } catch (failed_constructor& err) {
@@ -295,7 +304,6 @@ ARDOUR_UI::ARDOUR_UI (int *argcp, char **argvp[])
 
        keyboard = new ArdourKeyboard(*this);
 
-
        XMLNode* node = ARDOUR_UI::instance()->keyboard_settings();
        if (node) {
                keyboard->set_state (*node, Stateful::loading_state_version);
@@ -330,6 +338,11 @@ ARDOUR_UI::ARDOUR_UI (int *argcp, char **argvp[])
 
        starting.connect (sigc::mem_fun(*this, &ARDOUR_UI::startup));
        stopping.connect (sigc::mem_fun(*this, &ARDOUR_UI::shutdown));
+
+       _process_thread = new ProcessThread ();
+       _process_thread->init ();
+
+       DPIReset.connect (sigc::mem_fun (*this, &ARDOUR_UI::resize_text_widgets));
 }
 
 /** @return true if a session was chosen and `apply' clicked, otherwise false if `cancel' was clicked */
@@ -403,13 +416,22 @@ ARDOUR_UI::post_engine ()
 
        ARDOUR::init_post_engine ();
 
+       /* load up the UI manager */
+
        ActionManager::init ();
+
        _tooltips.enable();
 
        if (setup_windows ()) {
                throw failed_constructor ();
        }
 
+       /* Do this after setup_windows (), as that's when the _status_bar_visibility is created */
+       XMLNode* n = Config->extra_xml (X_("UI"));
+       if (n) {
+               _status_bar_visibility.set_state (*n);
+       }
+       
        check_memory_locking();
 
        /* this is the first point at which all the keybindings are available */
@@ -486,11 +508,10 @@ ARDOUR_UI::~ARDOUR_UI ()
 }
 
 void
-ARDOUR_UI::pop_back_splash ()
+ARDOUR_UI::pop_back_splash (Gtk::Window& win)
 {
        if (Splash::instance()) {
-               // Splash::instance()->pop_back();
-               Splash::instance()->hide ();
+               Splash::instance()->pop_back_for (win);
        }
 }
 
@@ -629,59 +650,6 @@ ARDOUR_UI::update_autosave ()
        }
 }
 
-void
-ARDOUR_UI::backend_audio_error (bool we_set_params, Gtk::Window* toplevel)
-{
-       string title;
-       if (we_set_params) {
-               title = string_compose (_("%1 could not start JACK"), PROGRAM_NAME);
-       } else {
-               title = string_compose (_("%1 could not connect to JACK."), PROGRAM_NAME);
-       }
-
-       MessageDialog win (title,
-                          false,
-                          Gtk::MESSAGE_INFO,
-                          Gtk::BUTTONS_NONE);
-
-       if (we_set_params) {
-               win.set_secondary_text(_("There are several possible reasons:\n\
-\n\
-1) You requested audio parameters that are not supported..\n\
-2) JACK is running as another user.\n\
-\n\
-Please consider the possibilities, and perhaps try different parameters."));
-       } else {
-               win.set_secondary_text(_("There are several possible reasons:\n\
-\n\
-1) JACK is not running.\n\
-2) JACK is running as another user, perhaps root.\n\
-3) There is already another client called \"ardour\".\n\
-\n\
-Please consider the possibilities, and perhaps (re)start JACK."));
-       }
-
-       if (toplevel) {
-               win.set_transient_for (*toplevel);
-       }
-
-       if (we_set_params) {
-               win.add_button (Stock::OK, RESPONSE_CLOSE);
-       } else {
-               win.add_button (Stock::QUIT, RESPONSE_CLOSE);
-       }
-
-       win.set_default_response (RESPONSE_CLOSE);
-
-       win.show_all ();
-       win.set_position (Gtk::WIN_POS_CENTER);
-       pop_back_splash ();
-
-       /* we just don't care about the result, but we want to block */
-
-       win.run ();
-}
-
 void
 ARDOUR_UI::startup ()
 {
@@ -713,6 +681,11 @@ ARDOUR_UI::startup ()
                add_window_proxy (_global_port_matrix[*i]);
        }
 
+       /* We have to do this here since goto_editor_window() ends up calling show_all() on the
+        * editor window, and we may want stuff to be hidden.
+        */
+       _status_bar_visibility.update ();
+
        BootMessage (string_compose (_("%1 is ready for use"), PROGRAM_NAME));
 }
 
@@ -774,7 +747,7 @@ ARDOUR_UI::check_memory_locking ()
                                vbox->show();
                                hbox.show ();
 
-                               pop_back_splash ();
+                               pop_back_splash (msg);
 
                                editor->ensure_float (msg);
                                msg.run ();
@@ -827,7 +800,7 @@ ARDOUR_UI::finish()
 Ardour was unable to save your session.\n\n\
 If you still wish to quit, please use the\n\n\
 \"Just quit\" option."));
-                                       pop_back_splash();
+                                       pop_back_splash(msg);
                                        msg.run ();
                                        return;
                                }
@@ -960,7 +933,7 @@ ARDOUR_UI::every_point_zero_one_seconds ()
 void
 ARDOUR_UI::update_sample_rate (framecnt_t)
 {
-       char buf[32];
+       char buf[64];
 
        ENSURE_GUI_THREAD (*this, &ARDOUR_UI::update_sample_rate, ignored)
 
@@ -973,40 +946,111 @@ ARDOUR_UI::update_sample_rate (framecnt_t)
                framecnt_t rate = engine->frame_rate();
 
                if (fmod (rate, 1000.0) != 0.0) {
-                       snprintf (buf, sizeof (buf), _("%.1f kHz / %4.1f ms"),
+                       snprintf (buf, sizeof (buf), _("JACK: <span foreground=\"green\">%.1f kHz / %4.1f ms</span>"),
                                  (float) rate/1000.0f,
                                  (engine->frames_per_cycle() / (float) rate) * 1000.0f);
                } else {
-                       snprintf (buf, sizeof (buf), _("%" PRId64 " kHz / %4.1f ms"),
+                       snprintf (buf, sizeof (buf), _("JACK: <span foreground=\"green\">%" PRId64 " kHz / %4.1f ms</span>"),
                                  rate/1000,
                                  (engine->frames_per_cycle() / (float) rate) * 1000.0f);
                }
        }
 
-       sample_rate_label.set_text (buf);
+       sample_rate_label.set_markup (buf);
+}
+
+void
+ARDOUR_UI::update_format ()
+{
+       if (!_session) {
+               format_label.set_text ("");
+               return;
+       }
+
+       stringstream s;
+       s << _("File:") << X_(" <span foreground=\"green\">");
+
+       switch (_session->config.get_native_file_header_format ()) {
+       case BWF:
+               s << _("BWF");
+               break;
+       case WAVE:
+               s << _("WAV");
+               break;
+       case WAVE64:
+               s << _("WAV64");
+               break;
+       case CAF:
+               s << _("CAF");
+               break;
+       case AIFF:
+               s << _("AIFF");
+               break;
+       case iXML:
+               s << _("iXML");
+               break;
+       case RF64:
+               s << _("RF64");
+               break;
+       }
+
+       s << " ";
+       
+       switch (_session->config.get_native_file_data_format ()) {
+       case FormatFloat:
+               s << _("32-float");
+               break;
+       case FormatInt24:
+               s << _("24-int");
+               break;
+       case FormatInt16:
+               s << _("16-int");
+               break;
+       }
+
+       s << X_("</span>");
+
+       format_label.set_markup (s.str ());
 }
 
 void
 ARDOUR_UI::update_cpu_load ()
 {
-       char buf[32];
-       snprintf (buf, sizeof (buf), _("DSP: %5.1f%%"), engine->get_cpu_load());
-       cpu_load_label.set_text (buf);
+       char buf[64];
+
+       /* If this text is changed, the set_size_request_to_display_given_text call in ARDOUR_UI::resize_text_widgets
+          should also be changed.
+       */
+
+       float const c = engine->get_cpu_load ();
+       snprintf (buf, sizeof (buf), _("DSP: <span foreground=\"%s\">%5.1f%%</span>"), c >= 90 ? X_("red") : X_("green"), c);
+       cpu_load_label.set_markup (buf);
 }
 
 void
 ARDOUR_UI::update_buffer_load ()
 {
-       char buf[64];
-       uint32_t c, p;
+       char buf[256];
+
+       uint32_t const playback = _session ? _session->playback_load () : 100;
+       uint32_t const capture = _session ? _session->capture_load () : 100;
 
+       /* If this text is changed, the set_size_request_to_display_given_text call in ARDOUR_UI::resize_text_widgets
+          should also be changed.
+       */
+       
        if (_session) {
-               c = _session->capture_load ();
-               p = _session->playback_load ();
+               snprintf (
+                       buf, sizeof (buf),
+                       _("Buffers: <span foreground=\"green\">p:</span><span foreground=\"%s\">%" PRIu32 "%%</span> "
+                                  "<span foreground=\"green\">c:</span><span foreground=\"%s\">%" PRIu32 "%%</span>"),
+                       playback <= 5 ? X_("red") : X_("green"),
+                       playback,
+                       capture <= 5 ? X_("red") : X_("green"),
+                       capture
+                       );
 
-               snprintf (buf, sizeof (buf), _("Buffers p:%" PRIu32 "%% c:%" PRIu32 "%%"),
-                         _session->playback_load(), _session->capture_load());
-               buffer_load_label.set_text (buf);
+               buffer_load_label.set_markup (buf);
        } else {
                buffer_load_label.set_text ("");
        }
@@ -1033,7 +1077,7 @@ ARDOUR_UI::update_disk_space()
        framecnt_t fr = _session->frame_rate();
 
        if (frames == max_framecnt) {
-               strcpy (buf, _("Disk: 24hrs+"));
+               snprintf (buf, sizeof (buf), _("Disk: <span foreground=\"green\">24hrs+</span>"));
        } else {
                rec_enabled_streams = 0;
                _session->foreach_route (this, &ARDOUR_UI::count_recenabled_streams);
@@ -1047,15 +1091,27 @@ ARDOUR_UI::update_disk_space()
                int secs;
 
                hrs  = frames / (fr * 3600);
-               frames -= hrs * fr * 3600;
-               mins = frames / (fr * 60);
-               frames -= mins * fr * 60;
-               secs = frames / fr;
 
-               snprintf (buf, sizeof(buf), _("Disk: %02dh:%02dm:%02ds"), hrs, mins, secs);
+               if (hrs > 24) {
+                       snprintf (buf, sizeof (buf), _("Disk: <span foreground=\"green\">&gt;24 hrs</span>"));
+               } else {
+                       frames -= hrs * fr * 3600;
+                       mins = frames / (fr * 60);
+                       frames -= mins * fr * 60;
+                       secs = frames / fr;
+                       
+                       bool const low = (hrs == 0 && mins <= 30);
+                       
+                       snprintf (
+                               buf, sizeof(buf),
+                               _("Disk: <span foreground=\"%s\">%02dh:%02dm:%02ds</span>"),
+                               low ? X_("red") : X_("green"),
+                               hrs, mins, secs
+                               );
+               }
        }
 
-       disk_space_label.set_text (buf);
+       disk_space_label.set_markup (buf);
 
        // An attempt to make the disk space label flash red when space has run out.
 
@@ -1083,13 +1139,6 @@ ARDOUR_UI::update_wall_clock ()
        return TRUE;
 }
 
-gint
-ARDOUR_UI::session_menu (GdkEventButton */*ev*/)
-{
-       session_popup_menu->popup (0, 0);
-       return TRUE;
-}
-
 void
 ARDOUR_UI::redisplay_recent_sessions ()
 {
@@ -1170,6 +1219,7 @@ ARDOUR_UI::redisplay_recent_sessions ()
                }
        }
 
+       recent_session_display.set_tooltip_column(1); // recent_session_columns.fullpath
        recent_session_display.set_model (recent_session_model);
 }
 
@@ -1271,7 +1321,7 @@ ARDOUR_UI::check_audioengine ()
                                                   _("%1 is not connected to JACK\n"
                                                     "You cannot open or close sessions in this condition"),
                                                   PROGRAM_NAME));
-                       pop_back_splash ();
+                       pop_back_splash (msg);
                        msg.run ();
                        return false;
                }
@@ -1299,6 +1349,17 @@ ARDOUR_UI::open_session ()
                open_session_selector->add_button (Gtk::Stock::CANCEL, Gtk::RESPONSE_CANCEL);
                open_session_selector->add_button (Gtk::Stock::OPEN, Gtk::RESPONSE_ACCEPT);
                open_session_selector->set_default_response(Gtk::RESPONSE_ACCEPT);
+               
+               if (_session) {
+                       string session_parent_dir = Glib::path_get_dirname(_session->path());
+                       string::size_type last_dir_sep = session_parent_dir.rfind(G_DIR_SEPARATOR);
+                       session_parent_dir = session_parent_dir.substr(0, last_dir_sep);
+                       open_session_selector->set_current_folder(session_parent_dir);
+               } else {
+                       open_session_selector->set_current_folder(Config->get_default_session_parent_dir());
+               }
+
+               open_session_selector->add_shortcut_folder (Config->get_default_session_parent_dir());
 
                FileFilter session_filter;
                session_filter.add_pattern ("*.ardour");
@@ -1333,7 +1394,7 @@ ARDOUR_UI::open_session ()
 
 
 void
-ARDOUR_UI::session_add_midi_route (bool disk, RouteGroup* route_group, uint32_t how_many, string const & name_template)
+ARDOUR_UI::session_add_midi_route (bool disk, RouteGroup* route_group, uint32_t how_many, const string& name_template, PluginInfoPtr instrument)
 {
        list<boost::shared_ptr<MidiTrack> > tracks;
 
@@ -1345,7 +1406,7 @@ ARDOUR_UI::session_add_midi_route (bool disk, RouteGroup* route_group, uint32_t
        try {
                if (disk) {
 
-                       tracks = _session->new_midi_track (ARDOUR::Normal, route_group, how_many, name_template);
+                       tracks = _session->new_midi_track (instrument, ARDOUR::Normal, route_group, how_many, name_template);
 
                        if (tracks.size() != how_many) {
                                if (how_many == 1) {
@@ -1354,6 +1415,7 @@ ARDOUR_UI::session_add_midi_route (bool disk, RouteGroup* route_group, uint32_t
                                        error << string_compose (_("could not create %1 new midi tracks"), how_many) << endmsg;
                                }
                        }
+                       
                } /*else {
                        if ((route = _session->new_midi_route ()) == 0) {
                                error << _("could not create new midi bus") << endmsg;
@@ -1410,9 +1472,9 @@ ARDOUR_UI::session_add_audio_route (
 
                        if (routes.size() != how_many) {
                                if (how_many == 1) {
-                                       error << _("could not create a new audio track") << endmsg;
+                                       error << _("could not create a new audio bus") << endmsg;
                                } else {
-                                       error << string_compose (_("could not create %1 new audio tracks"), how_many) << endmsg;
+                                       error << string_compose (_("could not create %1 new audio busses"), how_many) << endmsg;
                                }
                        }
                }
@@ -1424,30 +1486,11 @@ ARDOUR_UI::session_add_audio_route (
 to create a new track or bus.\n\
 You should save %1, exit and\n\
 restart JACK with more ports."), PROGRAM_NAME));
-               pop_back_splash ();
+               pop_back_splash (msg);
                msg.run ();
        }
 }
 
-void
-ARDOUR_UI::do_transport_locate (framepos_t new_position, bool with_roll)
-{
-       framecnt_t _preroll = 0;
-
-       if (_session) {
-               // XXX CONFIG_CHANGE FIX - requires AnyTime handling
-               // _preroll = _session->convert_to_frames_at (new_position, Config->get_preroll());
-
-               if (new_position > _preroll) {
-                       new_position -= _preroll;
-               } else {
-                       new_position = 0;
-               }
-
-               _session->request_locate (new_position, with_roll);
-       }
-}
-
 void
 ARDOUR_UI::transport_goto_start ()
 {
@@ -1540,22 +1583,6 @@ ARDOUR_UI::transport_stop ()
        _session->request_stop (false, true);
 }
 
-void
-ARDOUR_UI::transport_stop_and_forget_capture ()
-{
-       if (_session) {
-               _session->request_stop (true, true);
-       }
-}
-
-void
-ARDOUR_UI::remove_last_capture()
-{
-       if (editor) {
-               editor->remove_last_capture();
-       }
-}
-
 void
 ARDOUR_UI::transport_record (bool roll)
 {
@@ -1564,7 +1591,7 @@ ARDOUR_UI::transport_record (bool roll)
                switch (_session->record_status()) {
                case Session::Disabled:
                        if (_session->ntracks() == 0) {
-                               MessageDialog msg (*editor, _("Please create 1 or more track\nbefore trying to record.\nCheck the Session menu."));
+                               MessageDialog msg (*editor, _("Please create one or more tracks before trying to record.\nYou can do this with the \"Add Track or Bus\" option in the Session menu."));
                                msg.run ();
                                return;
                        }
@@ -1620,12 +1647,12 @@ ARDOUR_UI::transport_roll ()
                if (!Config->get_seamless_loop()) {
                        _session->request_play_loop (false, true);
                }
-       } else if (_session->get_play_range () && !join_play_range_button.get_active()) {
+       } else if (_session->get_play_range () && !Config->get_always_play_range()) {
                /* stop playing a range if we currently are */
                _session->request_play_range (0, true);
        }
 
-       if (join_play_range_button.get_active()) {
+       if (Config->get_always_play_range()) {
                _session->request_play_range (&editor->get_selection().time, true);
        }
 
@@ -1684,7 +1711,7 @@ ARDOUR_UI::toggle_roll (bool with_abort, bool roll_out_of_bounded_mode)
                if (rolling) {
                        _session->request_stop (with_abort, true);
                } else {
-                       if (join_play_range_button.get_active()) {
+                       if (Config->get_always_play_range ()) {
                                _session->request_play_range (&editor->get_selection().time, true);
                        }
 
@@ -1696,7 +1723,9 @@ ARDOUR_UI::toggle_roll (bool with_abort, bool roll_out_of_bounded_mode)
 void
 ARDOUR_UI::toggle_session_auto_loop ()
 {
-       if (!_session) {
+       Location * looploc = _session->locations()->auto_loop_location();
+
+       if (!_session || !looploc) {
                return;
        }
 
@@ -1704,24 +1733,18 @@ ARDOUR_UI::toggle_session_auto_loop ()
 
                if (_session->transport_rolling()) {
 
-                       Location * looploc = _session->locations()->auto_loop_location();
-
-                       if (looploc) {
-                               _session->request_locate (looploc->start(), true);
-                               _session->request_play_loop (false);
-                       }
+                       _session->request_locate (looploc->start(), true);
+                       _session->request_play_loop (false);
 
                } else {
                        _session->request_play_loop (false);
                }
        } else {
-
-         Location * looploc = _session->locations()->auto_loop_location();
-
-               if (looploc) {
-                       _session->request_play_loop (true);
-               }
+               _session->request_play_loop (true);
        }
+       
+       //show the loop markers
+       looploc->set_hidden (false, this);
 }
 
 void
@@ -1815,10 +1838,10 @@ void
 ARDOUR_UI::map_transport_state ()
 {
        if (!_session) {
-               auto_loop_button.set_visual_state (0);
-               play_selection_button.set_visual_state (0);
-               roll_button.set_visual_state (0);
-               stop_button.set_visual_state (1);
+               auto_loop_button.unset_active_state ();
+               play_selection_button.unset_active_state ();
+               roll_button.unset_active_state ();
+               stop_button.set_active_state (Gtkmm2ext::ExplicitActive);
                return;
        }
 
@@ -1832,37 +1855,37 @@ ARDOUR_UI::map_transport_state ()
 
                if (_session->get_play_range()) {
 
-                       play_selection_button.set_visual_state (1);
-                       roll_button.set_visual_state (0);
-                       auto_loop_button.set_visual_state (0);
+                       play_selection_button.set_active_state (Gtkmm2ext::ExplicitActive);
+                       roll_button.unset_active_state ();
+                       auto_loop_button.unset_active_state ();
 
                } else if (_session->get_play_loop ()) {
 
-                       auto_loop_button.set_visual_state (1);
-                       play_selection_button.set_visual_state (0);
-                       roll_button.set_visual_state (0);
+                       auto_loop_button.set_active (true);
+                       play_selection_button.set_active (false);
+                       roll_button.set_active (false);
 
                } else {
 
-                       roll_button.set_visual_state (1);
-                       play_selection_button.set_visual_state (0);
-                       auto_loop_button.set_visual_state (0);
+                       roll_button.set_active (true);
+                       play_selection_button.set_active (false);
+                       auto_loop_button.set_active (false);
                }
 
-               if (join_play_range_button.get_active()) {
+               if (Config->get_always_play_range()) {
                        /* light up both roll and play-selection if they are joined */
-                       roll_button.set_visual_state (1);
-                       play_selection_button.set_visual_state (1);
+                       roll_button.set_active (true);
+                       play_selection_button.set_active (true);
                }
 
-               stop_button.set_visual_state (0);
+               stop_button.set_active (false);
 
        } else {
 
-               stop_button.set_visual_state (1);
-               roll_button.set_visual_state (0);
-               play_selection_button.set_visual_state (0);
-               auto_loop_button.set_visual_state (0);
+               stop_button.set_active (true);
+               roll_button.set_active (false);
+               play_selection_button.set_active (false);
+               auto_loop_button.set_active (false);
                update_disk_space ();
        }
 }
@@ -1961,7 +1984,7 @@ JACK, reconnect and save the session."), PROGRAM_NAME);
        }
 
        MessageDialog msg (*editor, msgstr);
-       pop_back_splash ();
+       pop_back_splash (msg);
        msg.run ();
 
        if (free_reason) {
@@ -1987,12 +2010,6 @@ ARDOUR_UI::do_engine_start ()
        return 0;
 }
 
-void
-ARDOUR_UI::setup_theme ()
-{
-       theme_manager->setup_theme();
-}
-
 void
 ARDOUR_UI::update_clocks ()
 {
@@ -2013,18 +2030,6 @@ ARDOUR_UI::stop_clocking ()
        clock_signal_connection.disconnect ();
 }
 
-void
-ARDOUR_UI::toggle_clocking ()
-{
-#if 0
-       if (clock_button.get_active()) {
-               start_clocking ();
-       } else {
-               stop_clocking ();
-       }
-#endif
-}
-
 gint
 ARDOUR_UI::_blink (void *arg)
 
@@ -2062,7 +2067,7 @@ ARDOUR_UI::stop_blinking ()
 }
 
 
-/** Ask the user for the name of a new shapshot and then take it.
+/** Ask the user for the name of a new snapshot and then take it.
  */
 
 void
@@ -2074,7 +2079,6 @@ ARDOUR_UI::snapshot_session (bool switch_to_it)
        prompter.set_name ("Prompter");
        prompter.add_button (Gtk::Stock::SAVE, Gtk::RESPONSE_ACCEPT);
        prompter.set_title (_("Take Snapshot"));
-       prompter.set_title (_("Take Snapshot"));
        prompter.set_prompt (_("Name of new snapshot"));
 
        if (!switch_to_it) {
@@ -2084,7 +2088,7 @@ ARDOUR_UI::snapshot_session (bool switch_to_it)
 
                time (&n);
                localtime_r (&n, &local_time);
-               strftime (timebuf, sizeof(timebuf), "%FT%T", &local_time);
+               strftime (timebuf, sizeof(timebuf), "%FT%H.%M.%S", &local_time);
                prompter.set_initial_text (timebuf);
        }
 
@@ -2097,15 +2101,10 @@ ARDOUR_UI::snapshot_session (bool switch_to_it)
                bool do_save = (snapname.length() != 0);
 
                if (do_save) {
-                       if (snapname.find ('/') != string::npos) {
-                               MessageDialog msg (_("To ensure compatibility with various systems\n"
-                                                    "snapshot names may not contain a '/' character"));
-                               msg.run ();
-                               goto again;
-                       }
-                       if (snapname.find ('\\') != string::npos) {
-                               MessageDialog msg (_("To ensure compatibility with various systems\n"
-                                                    "snapshot names may not contain a '\\' character"));
+                       char illegal = Session::session_name_is_legal(snapname);
+                       if (illegal) {
+                               MessageDialog msg (string_compose (_("To ensure compatibility with various systems\n"
+                                                    "snapshot names may not contain a '%1' character"), illegal));
                                msg.run ();
                                goto again;
                        }
@@ -2139,6 +2138,69 @@ ARDOUR_UI::snapshot_session (bool switch_to_it)
        }
 }
 
+/** Ask the user for a new session name and then rename the session to it.
+ */
+
+void
+ARDOUR_UI::rename_session ()
+{
+       if (!_session) {
+               return;
+       }
+
+       ArdourPrompter prompter (true);
+       string name;
+
+       prompter.set_name ("Prompter");
+       prompter.add_button (Gtk::Stock::SAVE, Gtk::RESPONSE_ACCEPT);
+       prompter.set_title (_("Rename Session"));
+       prompter.set_prompt (_("New session name"));
+
+  again:
+       switch (prompter.run()) {
+       case RESPONSE_ACCEPT:
+       {
+               prompter.get_result (name);
+
+               bool do_rename = (name.length() != 0);
+
+               if (do_rename) {
+                       char illegal = Session::session_name_is_legal (name);
+
+                       if (illegal) {
+                               MessageDialog msg (string_compose (_("To ensure compatibility with various systems\n"
+                                                                    "session names may not contain a '%1' character"), illegal));
+                               msg.run ();
+                               goto again;
+                       }
+
+                       switch (_session->rename (name)) {
+                       case -1: {
+                               MessageDialog msg (_("That name is already in use by another directory/folder. Please try again."));
+                               msg.set_position (WIN_POS_MOUSE);
+                               msg.run ();
+                               goto again;
+                               break;
+                       }
+                       case 0:
+                               break;
+                       default: {
+                               MessageDialog msg (_("Renaming this session failed.\nThings could be seriously messed up at this point"));
+                               msg.set_position (WIN_POS_MOUSE);
+                               msg.run ();
+                               break;
+                       }
+                       }
+               }
+               
+               break;
+       }
+
+       default:
+               break;
+       }
+}
+
 void
 ARDOUR_UI::save_state (const string & name, bool switch_to_it)
 {
@@ -2150,6 +2212,8 @@ ARDOUR_UI::save_state (const string & name, bool switch_to_it)
                }
        }
 
+       node->add_child_nocopy (gui_object_state->get_state());
+
        _session->add_extra_xml (*node);
 
        save_state_canfail (name, switch_to_it);
@@ -2214,14 +2278,14 @@ ARDOUR_UI::transport_rec_enable_blink (bool onoff)
 
        if (r == Session::Enabled || (r == Session::Recording && !h)) {
                if (onoff) {
-                       rec_button.set_visual_state (2);
+                       rec_button.set_active_state (Gtkmm2ext::ExplicitActive);
                } else {
-                       rec_button.set_visual_state (0);
+                       rec_button.set_active_state (Gtkmm2ext::ImplicitActive);
                }
        } else if (r == Session::Recording && h) {
-               rec_button.set_visual_state (1);
+               rec_button.set_active_state (Gtkmm2ext::ExplicitActive);
        } else {
-               rec_button.set_visual_state (0);
+               rec_button.unset_active_state ();
        }
 }
 
@@ -2289,7 +2353,7 @@ ARDOUR_UI::ask_about_loading_existing_session (const std::string& session_path)
        msg.set_title (_("Open Existing Session"));
        msg.set_wmclass (X_("existing_session"), PROGRAM_NAME);
        msg.set_position (Gtk::WIN_POS_MOUSE);
-       pop_back_splash ();
+       pop_back_splash (msg);
 
        switch (msg.run()) {
        case RESPONSE_YES:
@@ -2328,8 +2392,6 @@ ARDOUR_UI::build_session_from_nsd (const std::string& session_path, const std::s
                        bus_profile.input_ac = AutoConnectOption (0);
                }
 
-               /// @todo some minor tweaks.
-
                bus_profile.output_ac = AutoConnectOption (0);
 
                if (_startup->connect_outputs ()) {
@@ -2387,17 +2449,17 @@ ARDOUR_UI::idle_load (const std::string& path)
 }
 
 void
-ARDOUR_UI::end_loading_messages ()
+ARDOUR_UI::loading_message (const std::string& msg)
 {
-       // hide_splash ();
-}
+       if (ARDOUR_COMMAND_LINE::no_splash) {
+               return;
+       }
 
-void
-ARDOUR_UI::loading_message (const std::string& /*msg*/)
-{
-       // show_splash ();
-       // splash->message (msg);
-       flush_pending ();
+       show_splash ();
+       if (splash) {
+               splash->message (msg);
+               flush_pending ();
+       }
 }
 
 /** @param quit_on_cancel true if exit() should be called if the user clicks `cancel' in the new session dialog */
@@ -2453,6 +2515,12 @@ ARDOUR_UI::get_session_parameters (bool quit_on_cancel, bool should_be_new, stri
 
                        session_name = _startup->session_name (likely_new);
 
+                       string::size_type suffix = session_name.find (statefile_suffix);
+
+                       if (suffix != string::npos) {
+                               session_name = session_name.substr (0, suffix);
+                       }
+
                        /* this shouldn't happen, but we catch it just in case it does */
 
                        if (session_name.empty()) {
@@ -2479,19 +2547,13 @@ ARDOUR_UI::get_session_parameters (bool quit_on_cancel, bool should_be_new, stri
 
                                session_path = _startup->session_folder();
 
-                               if (session_name.find ('/') != string::npos) {
-                                       MessageDialog msg (*_startup,
-                                                          _("To ensure compatibility with various systems\n"
-                                                            "session names may not contain a '/' character"));
-                                       msg.run ();
-                                       ARDOUR_COMMAND_LINE::session_name = ""; // cancel that
-                                       continue;
-                               }
+                               char illegal = Session::session_name_is_legal (session_name);
 
-                               if (session_name.find ('\\') != string::npos) {
+                               if (illegal) {
                                        MessageDialog msg (*_startup,
-                                                          _("To ensure compatibility with various systems\n"
-                                                            "session names may not contain a '\\' character"));
+                                                          string_compose (_("To ensure compatibility with various systems\n"
+                                                                            "session names may not contain a '%1' character"),
+                                                                          illegal));
                                        msg.run ();
                                        ARDOUR_COMMAND_LINE::session_name = ""; // cancel that
                                        continue;
@@ -2520,25 +2582,23 @@ ARDOUR_UI::get_session_parameters (bool quit_on_cancel, bool should_be_new, stri
                } else {
 
                        if (!likely_new) {
-                               MessageDialog msg (string_compose (_("There is no existing session at \"%1\""), session_path));
-                               msg.run ();
-                               ARDOUR_COMMAND_LINE::session_name = ""; // cancel that
-                               continue;
-                       }
+                               if (_startup) {
+                                       pop_back_splash (*_startup);
+                               } else {
+                                       hide_splash ();
+                               }
 
-                       if (session_name.find ('/') != std::string::npos) {
-                               MessageDialog msg (*_startup,
-                                                  _("To ensure compatibility with various systems\n"
-                                                    "session names may not contain a '/' character"));
+                               MessageDialog msg (string_compose (_("There is no existing session at \"%1\""), session_path));
                                msg.run ();
                                ARDOUR_COMMAND_LINE::session_name = ""; // cancel that
                                continue;
                        }
 
-                       if (session_name.find ('\\') != std::string::npos) {
-                               MessageDialog msg (*_startup,
-                                                  _("To ensure compatibility with various systems\n"
-                                                    "session names may not contain a '\\' character"));
+                       char illegal = Session::session_name_is_legal(session_name);
+                       if (illegal) {
+                               pop_back_splash (*_startup);
+                               MessageDialog msg (*_startup, string_compose(_("To ensure compatibility with various systems\n"
+                                                    "session names may not contain a '%1' character"), illegal));
                                msg.run ();
                                ARDOUR_COMMAND_LINE::session_name = ""; // cancel that
                                continue;
@@ -2633,7 +2693,7 @@ ARDOUR_UI::load_session (const std::string& path, const std::string& snap_name,
                msg.set_title (_("Port Registration Error"));
                msg.set_secondary_text (_("Click the Close button to try again."));
                msg.set_position (Gtk::WIN_POS_CENTER);
-               pop_back_splash ();
+               pop_back_splash (msg);
                msg.present ();
 
                int response = msg.run ();
@@ -2662,7 +2722,7 @@ ARDOUR_UI::load_session (const std::string& path, const std::string& snap_name,
                msg.set_secondary_text (_("Click the Refresh button to try again."));
                msg.add_button (Stock::REFRESH, 1);
                msg.set_position (Gtk::WIN_POS_CENTER);
-               pop_back_splash ();
+               pop_back_splash (msg);
                msg.present ();
 
                int response = msg.run ();
@@ -2742,7 +2802,7 @@ ARDOUR_UI::build_session (const std::string& path, const std::string& snap_name,
        catch (...) {
 
                MessageDialog msg (string_compose(_("Could not create session in \"%1\""), path));
-               pop_back_splash ();
+               pop_back_splash (msg);
                msg.run ();
                return -1;
        }
@@ -2831,12 +2891,13 @@ ARDOUR_UI::show_splash ()
                try {
                        splash = new Splash;
                } catch (...) {
+                       cerr << "Splash could not be created\n";
                        return;
                }
        }
 
-       splash->show ();
        splash->present ();
+       splash->pop_front ();
        splash->queue_draw ();
        splash->get_window()->process_updates (true);
        flush_pending ();
@@ -2863,7 +2924,7 @@ ARDOUR_UI::display_cleanup_results (ARDOUR::CleanupReport& rep, const gchar* lis
                                    _("No files were ready for clean-up"),
                                    true,
                                    Gtk::MESSAGE_INFO,
-                                   (Gtk::ButtonsType)(Gtk::BUTTONS_OK)  );
+                                   Gtk::BUTTONS_OK);
                msgd.set_title (_("Clean-up"));
                msgd.set_secondary_text (_("If this seems suprising, \n\
 check for any existing snapshots.\n\
@@ -2935,9 +2996,9 @@ require some unused files to continue to exist."));
        }
 
        if (removed > 1) {
-               txt.set_text (string_compose (plural_msg, removed, dead_directory, space_adjusted, bprefix));
+               txt.set_text (string_compose (plural_msg, removed, dead_directory, space_adjusted, bprefix, PROGRAM_NAME));
        } else {
-               txt.set_text (string_compose (singular_msg, removed, dead_directory, space_adjusted, bprefix));
+               txt.set_text (string_compose (singular_msg, removed, dead_directory, space_adjusted, bprefix, PROGRAM_NAME));
        }
 
        dhbox.pack_start (*dimage, true, false, 5);
@@ -2989,7 +3050,7 @@ ARDOUR_UI::cleanup ()
        MessageDialog checker (_("Are you sure you want to clean-up?"),
                                true,
                                Gtk::MESSAGE_QUESTION,
-                               (Gtk::ButtonsType)(Gtk::BUTTONS_NONE));
+                               Gtk::BUTTONS_NONE);
 
        checker.set_title (_("Clean-up"));
 
@@ -3037,7 +3098,7 @@ Clean-up will move all unused files to a \"dead\" location."));
 The following %1 files were not in use and \n\
 have been moved to:\n\n\
 %2\n\n\
-After a restart of Ardour,\n\n\
+After a restart of %5,\n\n\
 Session -> Clean-up -> Flush Wastebasket\n\n\
 will release an additional\n\
 %3 %4bytes of disk space.\n"),
@@ -3045,7 +3106,7 @@ will release an additional\n\
 The following file was not in use and \n\
 has been moved to:\n                           \
 %2\n\n\
-After a restart of Ardour,\n\n\
+After a restart of %5,\n\n\
 Session -> Clean-up -> Flush Wastebasket\n\n\
 will release an additional\n\
 %3 %4bytes of disk space.\n"
@@ -3088,6 +3149,7 @@ ARDOUR_UI::add_route (Gtk::Window* float_window)
 
        if (add_route_dialog == 0) {
                add_route_dialog = new AddRouteDialog (_session);
+               add_route_dialog->set_position (WIN_POS_MOUSE);
                if (float_window) {
                        add_route_dialog->set_transient_for (*float_window);
                }
@@ -3124,7 +3186,7 @@ ARDOUR_UI::add_route (Gtk::Window* float_window)
        uint32_t input_chan = add_route_dialog->channels ();
        uint32_t output_chan;
        string name_template = add_route_dialog->name_template ();
-       bool track = add_route_dialog->track ();
+       PluginInfoPtr instrument = add_route_dialog->requested_instrument ();
        RouteGroup* route_group = add_route_dialog->route_group ();
 
        AutoConnectOption oac = Config->get_output_auto_connect();
@@ -3137,21 +3199,12 @@ ARDOUR_UI::add_route (Gtk::Window* float_window)
 
        /* XXX do something with name template */
 
-       if (add_route_dialog->type() == ARDOUR::DataType::MIDI) {
-               if (track) {
-                       session_add_midi_track (route_group, count, name_template);
-               } else  {
-                       MessageDialog msg (*editor,
-                                       _("Sorry, MIDI Busses are not supported at this time."));
-                       msg.run ();
-                       //session_add_midi_bus();
-               }
+       if (add_route_dialog->midi_tracks_wanted()) {
+               session_add_midi_track (route_group, count, name_template, instrument);
+       } else if (add_route_dialog->audio_tracks_wanted()) {
+               session_add_audio_track (input_chan, output_chan, add_route_dialog->mode(), route_group, count, name_template);
        } else {
-               if (track) {
-                       session_add_audio_track (input_chan, output_chan, add_route_dialog->mode(), route_group, count, name_template);
-               } else {
-                       session_add_audio_bus (input_chan, output_chan, route_group, count, name_template);
-               }
+               session_add_audio_bus (input_chan, output_chan, route_group, count, name_template);
        }
 }
 
@@ -3409,13 +3462,13 @@ void
 ARDOUR_UI::update_transport_clocks (framepos_t pos)
 {
        if (Config->get_primary_clock_delta_edit_cursor()) {
-               primary_clock->set (pos, false, editor->get_preferred_edit_position(), 1);
+               primary_clock->set (pos, false, editor->get_preferred_edit_position());
        } else {
-               primary_clock->set (pos, 0, true);
+               primary_clock->set (pos);
        }
 
        if (Config->get_secondary_clock_delta_edit_cursor()) {
-               secondary_clock->set (pos, false, editor->get_preferred_edit_position(), 2);
+               secondary_clock->set (pos, false, editor->get_preferred_edit_position());
        } else {
                secondary_clock->set (pos);
        }
@@ -3433,10 +3486,10 @@ ARDOUR_UI::step_edit_status_change (bool yn)
        // we make insensitive
 
        if (yn) {
-               rec_button.set_visual_state (3);
+               rec_button.set_active_state (Gtkmm2ext::ImplicitActive);
                rec_button.set_sensitive (false);
        } else {
-               rec_button.set_visual_state (0);
+               rec_button.unset_active_state ();;
                rec_button.set_sensitive (true);
        }
 }
@@ -3451,13 +3504,10 @@ ARDOUR_UI::record_state_changed ()
                return;
        }
 
-       Session::RecordState const r = _session->record_status ();
-       bool const h = _session->have_rec_enabled_track ();
-
-       if (r == Session::Recording && h)  {
-               big_clock->set_widget_name ("BigClockRecording");
+       if (_session->record_status () == Session::Recording && _session->have_rec_enabled_track ()) {
+               big_clock->set_active (true);
        } else {
-               big_clock->set_widget_name ("BigClockNonRecording");
+               big_clock->set_active (false);
        }
 }
 
@@ -3482,15 +3532,19 @@ ARDOUR_UI::store_clock_modes ()
        XMLNode* node = new XMLNode(X_("ClockModes"));
 
        for (vector<AudioClock*>::iterator x = AudioClock::clocks.begin(); x != AudioClock::clocks.end(); ++x) {
-               node->add_property ((*x)->name().c_str(), enum_2_string ((*x)->mode()));
+               XMLNode* child = new XMLNode (X_("Clock"));
+               
+               child->add_property (X_("name"), (*x)->name());
+               child->add_property (X_("mode"), enum_2_string ((*x)->mode()));
+               child->add_property (X_("on"), ((*x)->off() ? X_("no") : X_("yes")));
+
+               node->add_child_nocopy (*child);
        }
 
        _session->add_extra_xml (*node);
        _session->set_dirty ();
 }
 
-
-
 ARDOUR_UI::TransportControllable::TransportControllable (std::string name, ARDOUR_UI& u, ToggleType tp)
        : Controllable (name), ui (u), type(tp)
 {
@@ -3571,12 +3625,6 @@ ARDOUR_UI::TransportControllable::get_value (void) const
        return val;
 }
 
-void
-ARDOUR_UI::TransportControllable::set_id (const string& str)
-{
-       _id = str;
-}
-
 void
 ARDOUR_UI::setup_profile ()
 {
@@ -3672,7 +3720,7 @@ ARDOUR_UI::missing_file (Session*s, std::string str, DataType type)
 }
 
 int
-ARDOUR_UI::ambiguous_file (std::string file, std::string path, std::vector<std::string> hits)
+ARDOUR_UI::ambiguous_file (std::string file, std::string /*path*/, std::vector<std::string> hits)
 {
        AmbiguousFileDialog dialog (file, hits);
 
@@ -3682,3 +3730,37 @@ ARDOUR_UI::ambiguous_file (std::string file, std::string path, std::vector<std::
        dialog.run ();
        return dialog.get_which ();
 }
+
+/** Allocate our thread-local buffers */
+void
+ARDOUR_UI::get_process_buffers ()
+{
+       _process_thread->get_buffers ();
+}
+
+/** Drop our thread-local buffers */
+void
+ARDOUR_UI::drop_process_buffers ()
+{
+       _process_thread->drop_buffers ();
+}
+
+void
+ARDOUR_UI::feedback_detected ()
+{
+       _feedback_exists = true;
+}
+
+void
+ARDOUR_UI::successful_graph_sort ()
+{
+       _feedback_exists = false;
+}
+
+void
+ARDOUR_UI::midi_panic ()
+{
+       if (_session) {
+               _session->midi_panic();
+       }
+}