Rearrange condition for pure midi plugins
[ardour.git] / gtk2_ardour / ardour_ui_options.cc
index 0e9f193653d6306ab174aa768156c3f640cbad06..98ede1dc3fbcd1742264fad39eee1aa3a8e3bd44 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2005 Paul Davis 
+    Copyright (C) 2005 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
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 
-    $Id$
 */
 
-#include <pbd/convert.h>
+#ifdef WAF_BUILD
+#include "gtk2ardour-config.h"
+#endif
+
+#include "pbd/convert.h"
+#include "pbd/stacktrace.h"
 
 #include <gtkmm2ext/utils.h>
 
-#include <ardour/configuration.h>
-#include <ardour/session.h>
-#include <ardour/audioengine.h>
+#include "ardour/rc_configuration.h"
+#include "ardour/session.h"
+
+#include "canvas/wave_view.h"
 
+#include "audio_clock.h"
 #include "ardour_ui.h"
 #include "actions.h"
 #include "gui_thread.h"
+#include "public_editor.h"
+#include "main_clock.h"
 
-#include "i18n.h"
+#include "pbd/i18n.h"
 
 using namespace Gtk;
 using namespace Gtkmm2ext;
 using namespace ARDOUR;
 using namespace PBD;
-using namespace sigc;
 
 void
-ARDOUR_UI::toggle_config_state (const char* group, const char* action, bool (Configuration::*set)(bool), bool (Configuration::*get)(void) const)
+ARDOUR_UI::toggle_external_sync()
 {
-       Glib::RefPtr<Action> act = ActionManager::get_action (group, action);
-       if (act) {
-               Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
-
-               if (tact) {
-                       bool x = (Config->*get)();
-                       
-                       if (x != tact->get_active()) {
-                               (Config->*set) (!x);
+       if (_session) {
+               if (_session->config.get_video_pullup() != 0.0f) {
+                       if (Config->get_sync_source() == Engine) {
+                               MessageDialog msg (
+                                       _("It is not possible to use JACK as the the sync source\n\
+when the pull up/down setting is non-zero."));
+                               msg.run ();
+                               return;
                        }
                }
-       }
-}
 
-void
-ARDOUR_UI::toggle_config_state (const char* group, const char* action, sigc::slot<void> theSlot)
-{
-       if (session) {
-               Glib::RefPtr<Action> act = ActionManager::get_action (group, action);
-               if (act) {
-                       Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
-                       if (tact->get_active()) {
-                               theSlot ();
-                       }
-               }
+               ActionManager::toggle_config_state_foo ("Transport", "ToggleExternalSync", sigc::mem_fun (_session->config, &SessionConfiguration::set_external_sync), sigc::mem_fun (_session->config, &SessionConfiguration::get_external_sync));
+
+               /* activating a slave is a session-property.
+                * The slave type is a RC property.
+                * When the slave is active is must not be reconfigured.
+                * This is a UI limitation, imposed by audio-clock and
+                * status displays which combine RC-config & session-properties.
+                *
+                * Notficy RCOptionEditor by emitting a signal if the active
+                * status changed:
+                */
+               Config->ParameterChanged("sync-source");
        }
 }
 
 void
 ARDOUR_UI::toggle_time_master ()
 {
-       toggle_config_state ("Transport", "ToggleTimeMaster", &Configuration::set_jack_time_master, &Configuration::get_jack_time_master);
-       if (session) {
-               session->engine().reset_timebase ();
-       }
+       ActionManager::toggle_config_state_foo ("Transport", "ToggleTimeMaster", sigc::mem_fun (_session->config, &SessionConfiguration::set_jack_time_master), sigc::mem_fun (_session->config, &SessionConfiguration::get_jack_time_master));
 }
 
 void
 ARDOUR_UI::toggle_send_mtc ()
 {
-       toggle_config_state ("options", "SendMTC", &Configuration::set_send_mtc, &Configuration::get_send_mtc);
+       ActionManager::toggle_config_state ("Options", "SendMTC", &RCConfiguration::set_send_mtc, &RCConfiguration::get_send_mtc);
 }
 
 void
 ARDOUR_UI::toggle_send_mmc ()
 {
-       toggle_config_state ("options", "SendMMC", &Configuration::set_send_mmc, &Configuration::get_send_mmc);
+       ActionManager::toggle_config_state ("Options", "SendMMC", &RCConfiguration::set_send_mmc, &RCConfiguration::get_send_mmc);
 }
 
 void
-ARDOUR_UI::toggle_use_mmc ()
+ARDOUR_UI::toggle_send_midi_clock ()
 {
-       toggle_config_state ("options", "UseMMC", &Configuration::set_mmc_control, &Configuration::get_mmc_control);
+       ActionManager::toggle_config_state ("Options", "SendMidiClock", &RCConfiguration::set_send_midi_clock, &RCConfiguration::get_send_midi_clock);
 }
 
 void
-ARDOUR_UI::toggle_use_midi_control ()
+ARDOUR_UI::toggle_use_mmc ()
 {
-       toggle_config_state ("options", "UseMIDIcontrol", &Configuration::set_midi_control, &Configuration::get_midi_control);
+       ActionManager::toggle_config_state ("Options", "UseMMC", &RCConfiguration::set_mmc_control, &RCConfiguration::get_mmc_control);
 }
 
 void
 ARDOUR_UI::toggle_send_midi_feedback ()
 {
-       toggle_config_state ("options", "SendMIDIfeedback", &Configuration::set_midi_feedback, &Configuration::get_midi_feedback);
-}
-
-void
-ARDOUR_UI::toggle_AutoConnectNewTrackInputsToHardware()
-{
-       toggle_config_state ("options", "AutoConnectNewTrackInputsToHardware", hide_return (bind (mem_fun (*Config, &Configuration::set_input_auto_connect), AutoConnectPhysical)));
-}
-void
-ARDOUR_UI::toggle_AutoConnectNewTrackOutputsToHardware()
-{
-       toggle_config_state ("options", "AutoConnectNewTrackOutputsToHardware", hide_return (bind (mem_fun (*Config, &Configuration::set_output_auto_connect), AutoConnectPhysical)));
-}
-void
-ARDOUR_UI::toggle_AutoConnectNewTrackOutputsToMaster()
-{
-       toggle_config_state ("options", "AutoConnectNewTrackOutputsToHardware", hide_return (bind (mem_fun (*Config, &Configuration::set_output_auto_connect), AutoConnectMaster)));
-}
-void
-ARDOUR_UI::toggle_ManuallyConnectNewTrackOutputs()
-{
-       toggle_config_state ("options", "AutoConnectNewTrackOutputsToHardware", hide_return (bind (mem_fun (*Config, &Configuration::set_output_auto_connect), AutoConnectOption (0))));
+       ActionManager::toggle_config_state ("Options", "SendMIDIfeedback", &RCConfiguration::set_midi_feedback, &RCConfiguration::get_midi_feedback);
 }
 
 void
 ARDOUR_UI::toggle_auto_input ()
 {
-       toggle_config_state ("Transport", "ToggleAutoInput", &Configuration::set_auto_input, &Configuration::get_auto_input);
+       ActionManager::toggle_config_state_foo ("Transport", "ToggleAutoInput", sigc::mem_fun (_session->config, &SessionConfiguration::set_auto_input), sigc::mem_fun (_session->config, &SessionConfiguration::get_auto_input));
 }
 
 void
 ARDOUR_UI::toggle_auto_play ()
 {
-       toggle_config_state ("Transport", "ToggleAutoPlay", &Configuration::set_auto_play, &Configuration::get_auto_play);
+       ActionManager::toggle_config_state_foo ("Transport", "ToggleAutoPlay", sigc::mem_fun (_session->config, &SessionConfiguration::set_auto_play), sigc::mem_fun (_session->config, &SessionConfiguration::get_auto_play));
 }
 
 void
 ARDOUR_UI::toggle_auto_return ()
 {
-       toggle_config_state ("Transport", "ToggleAutoReturn", &Configuration::set_auto_return, &Configuration::get_auto_return);
+       ActionManager::toggle_config_state_foo ("Transport", "ToggleAutoReturn", sigc::mem_fun (_session->config, &SessionConfiguration::set_auto_return), sigc::mem_fun (_session->config, &SessionConfiguration::get_auto_return));
 }
 
 void
 ARDOUR_UI::toggle_click ()
 {
-       toggle_config_state ("Transport", "ToggleClick", &Configuration::set_clicking, &Configuration::get_clicking);
+       ActionManager::toggle_config_state ("Transport", "ToggleClick", &RCConfiguration::set_clicking, &RCConfiguration::get_clicking);
 }
 
 void
-ARDOUR_UI::toggle_session_auto_loop ()
+ARDOUR_UI::unset_dual_punch ()
 {
-       if (session) {
-               if (Config->get_auto_loop()) {
-                       if (session->transport_rolling()) {
-                               transport_roll();
-                       } else {
-                               session->request_play_loop (false);
-                       }
-               } else {
-                       session->request_play_loop (true);
+       Glib::RefPtr<Action> action = ActionManager::get_action ("Transport", "TogglePunch");
+
+       if (action) {
+               Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(action);
+               if (tact) {
+                       ignore_dual_punch = true;
+                       tact->set_active (false);
+                       ignore_dual_punch = false;
                }
        }
 }
 
 void
-ARDOUR_UI::toggle_punch_in ()
+ARDOUR_UI::toggle_punch ()
 {
-       toggle_config_state ("Transport", "TogglePunchIn", &Configuration::set_punch_in, &Configuration::get_punch_in);
-}
+       if (ignore_dual_punch) {
+               return;
+       }
 
-void
-ARDOUR_UI::toggle_punch_out ()
-{
-       toggle_config_state ("Transport", "TogglePunchOut", &Configuration::set_punch_out, &Configuration::get_punch_out);
-}
+       Glib::RefPtr<Action> action = ActionManager::get_action ("Transport", "TogglePunch");
 
-void
-ARDOUR_UI::toggle_video_sync()
-{
-       Glib::RefPtr<Action> act = ActionManager::get_action ("Transport", "ToggleVideoSync");
-       if (act) {
-               Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
-               Config->set_use_video_sync (tact->get_active());
-       }
-}
+       if (action) {
 
-void
-ARDOUR_UI::toggle_editing_space()
-{
-       Glib::RefPtr<Action> act = ActionManager::get_action ("Common", "ToggleMaximalEditor");
-       if (act) {
-               Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
-               if (tact->get_active()) {
-                       maximise_editing_space ();
-               } else {
-                       restore_editing_space ();
+               Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(action);
+
+               if (!tact) {
+                       return;
                }
-       }
-}
 
-void
-ARDOUR_UI::toggle_UseHardwareMonitoring()
-{
-       Glib::RefPtr<Action> act = ActionManager::get_action ("options", "UseHardwareMonitoring");
-       if (act) {
-               Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
-               if (tact->get_active()) {
-                       Config->set_use_hardware_monitoring (true);
-                       Config->set_use_sw_monitoring (false);
-                       Config->set_use_external_monitoring (false);
-                       if (session) {
-                               session->reset_input_monitor_state();
-                       }
+               /* drive the other two actions from this one */
+
+               Glib::RefPtr<Action> in_action = ActionManager::get_action ("Transport", "TogglePunchIn");
+               Glib::RefPtr<Action> out_action = ActionManager::get_action ("Transport", "TogglePunchOut");
+
+               if (in_action && out_action) {
+                       Glib::RefPtr<ToggleAction> tiact = Glib::RefPtr<ToggleAction>::cast_dynamic(in_action);
+                       Glib::RefPtr<ToggleAction> toact = Glib::RefPtr<ToggleAction>::cast_dynamic(out_action);
+                       tiact->set_active (tact->get_active());
+                       toact->set_active (tact->get_active());
                }
        }
 }
 
 void
-ARDOUR_UI::toggle_UseSoftwareMonitoring()
+ARDOUR_UI::toggle_punch_in ()
 {
-       Glib::RefPtr<Action> act = ActionManager::get_action ("options", "UseSoftwareMonitoring");
-       if (act) {
-               Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
-               if (tact->get_active()) {
-                       Config->set_use_hardware_monitoring (false);
-                       Config->set_use_sw_monitoring (true);
-                       Config->set_use_external_monitoring (false);
-                       if (session) {
-                               session->reset_input_monitor_state();
-                       }
-               }
+       Glib::RefPtr<Action> act = ActionManager::get_action (X_("Transport"), X_("TogglePunchIn"));
+       if (!act) {
+               return;
        }
-}
 
-void
-ARDOUR_UI::toggle_UseExternalMonitoring()
-{
-       Glib::RefPtr<Action> act = ActionManager::get_action ("options", "UseExternalMonitoring");
-       if (act) {
-               Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
-               if (tact->get_active()) {
-                       Config->set_use_hardware_monitoring (false);
-                       Config->set_use_sw_monitoring (false);
-                       Config->set_use_external_monitoring (true);
-                       if (session) {
-                               session->reset_input_monitor_state();
-                       }
-               }
+       Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic (act);
+       if (!tact) {
+               return;
        }
-}
 
-void
-ARDOUR_UI::toggle_StopPluginsWithTransport()
-{
-       toggle_config_state ("options", "StopPluginsWithTransport", &Configuration::set_plugins_stop_with_transport, &Configuration::get_plugins_stop_with_transport);
-}
+       if (tact->get_active() != _session->config.get_punch_in()) {
+               _session->config.set_punch_in (tact->get_active ());
+       }
 
-void
-ARDOUR_UI::toggle_LatchedRecordEnable()
-{
-       toggle_config_state ("options", "LatchedRecordEnable", &Configuration::set_latched_record_enable, &Configuration::get_latched_record_enable);
+       if (tact->get_active()) {
+               /* if punch-in is turned on, make sure the loop/punch ruler is visible, and stop it being hidden,
+                  to avoid confusing the user */
+               show_loop_punch_ruler_and_disallow_hide ();
+       }
+
+       reenable_hide_loop_punch_ruler_if_appropriate ();
 }
 
 void
-ARDOUR_UI::toggle_DoNotRunPluginsWhileRecording()
+ARDOUR_UI::toggle_punch_out ()
 {
-       toggle_config_state ("options", "DoNotRunPluginsWhileRecording", &Configuration::set_do_not_record_plugins, &Configuration::get_do_not_record_plugins);
+       Glib::RefPtr<Action> act = ActionManager::get_action (X_("Transport"), X_("TogglePunchOut"));
+       if (!act) {
+               return;
+       }
+
+       Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic (act);
+       if (!tact) {
+               return;
+       }
+
+       if (tact->get_active() != _session->config.get_punch_out()) {
+               _session->config.set_punch_out (tact->get_active ());
+       }
+
+       if (tact->get_active()) {
+               /* if punch-out is turned on, make sure the loop/punch ruler is visible, and stop it being hidden,
+                  to avoid confusing the user */
+               show_loop_punch_ruler_and_disallow_hide ();
+       }
+
+       reenable_hide_loop_punch_ruler_if_appropriate ();
 }
 
 void
-ARDOUR_UI::toggle_VerifyRemoveLastCapture()
+ARDOUR_UI::show_loop_punch_ruler_and_disallow_hide ()
 {
-       toggle_config_state ("options", "VerifyRemoveLastCapture", &Configuration::set_verify_remove_last_capture, &Configuration::get_verify_remove_last_capture);
+       Glib::RefPtr<Action> act = ActionManager::get_action (X_("Rulers"), "toggle-loop-punch-ruler");
+       if (!act) {
+               return;
+       }
+
+       act->set_sensitive (false);
+
+       Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic (act);
+       if (!tact) {
+               return;
+       }
+
+       if (!tact->get_active()) {
+               tact->set_active ();
+       }
 }
 
+/* This is a bit of a silly name for a method */
 void
-ARDOUR_UI::toggle_StopRecordingOnXrun()
+ARDOUR_UI::reenable_hide_loop_punch_ruler_if_appropriate ()
 {
-       toggle_config_state ("options", "StopRecordingOnXrun", &Configuration::set_stop_recording_on_xrun, &Configuration::get_stop_recording_on_xrun);
+       if (!_session->config.get_punch_in() && !_session->config.get_punch_out()) {
+               /* if punch in/out are now both off, reallow hiding of the loop/punch ruler */
+               Glib::RefPtr<Action> act = ActionManager::get_action (X_("Rulers"), "toggle-loop-punch-ruler");
+               if (act) {
+                       act->set_sensitive (true);
+               }
+       }
 }
 
 void
-ARDOUR_UI::toggle_StopTransportAtEndOfSession()
+ARDOUR_UI::toggle_video_sync()
 {
-       toggle_config_state ("options", "StopTransportAtEndOfSession", &Configuration::set_stop_at_session_end, &Configuration::get_stop_at_session_end);
+       Glib::RefPtr<Action> act = ActionManager::get_action ("Transport", "ToggleVideoSync");
+       if (act) {
+               Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
+               _session->config.set_use_video_sync (tact->get_active());
+       }
 }
 
 void
-ARDOUR_UI::toggle_GainReduceFastTransport()
+ARDOUR_UI::toggle_editing_space()
 {
-       Glib::RefPtr<Action> act = ActionManager::get_action ("options", "GainReduceFastTransport");
+       Glib::RefPtr<Action> act = ActionManager::get_action ("Common", "ToggleMaximalEditor");
+
        if (act) {
                Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
                if (tact->get_active()) {
-                       Config->set_quieten_at_speed (0.251189); // -12dB reduction for ffwd or rewind
+                       maximise_editing_space ();
                } else {
-                       Config->set_quieten_at_speed (1.0); /* no change */
+                       restore_editing_space ();
                }
        }
 }
 
 void
-ARDOUR_UI::toggle_LatchedSolo()
+ARDOUR_UI::setup_session_options ()
 {
-       toggle_config_state ("options", "LatchedSolo", &Configuration::set_solo_latched, &Configuration::get_solo_latched);
+       _session->config.ParameterChanged.connect (_session_connections, MISSING_INVALIDATOR, boost::bind (&ARDOUR_UI::parameter_changed, this, _1), gui_context());
+       boost::function<void (std::string)> pc (boost::bind (&ARDOUR_UI::parameter_changed, this, _1));
+       _session->config.map_parameters (pc);
 }
 
 void
-ARDOUR_UI::toggle_SoloViaBus()
+ARDOUR_UI::parameter_changed (std::string p)
 {
-       if (!session) {
-               return;
-       }
+       if (p == "external-sync") {
 
-       Glib::RefPtr<Action> act = ActionManager::get_action ("options", "SoloViaBus");
-       if (act) {
-               Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
+               ActionManager::map_some_state ("Transport", "ToggleExternalSync", sigc::mem_fun (_session->config, &SessionConfiguration::get_external_sync));
 
-               if (tact->get_active()) {
-                       Config->set_solo_model (SoloBus);
+               if (!_session->config.get_external_sync()) {
+                       sync_button.set_text (_("Internal"));
+                       auto_loop_button.set_sensitive (true);
+                       ActionManager::get_action ("Transport", "ToggleAutoPlay")->set_sensitive (true);
+                       ActionManager::get_action ("Transport", "ToggleAutoReturn")->set_sensitive (true);
+                       ActionManager::get_action ("Transport", "ToggleFollowEdits")->set_sensitive (true);
                } else {
-                       Config->set_solo_model (InverseMute);
+                       sync_button.set_text (sync_source_to_string (Config->get_sync_source(), true));
+                       if (_session && _session->locations()->auto_loop_location()) {
+                               // disable looping with external sync.
+                               // This is not necessary because session-transport ignores the loop-state,
+                               // but makes it clear to the user that it's disabled.
+                               _session->request_play_loop (false, false);
+                       }
+                       auto_loop_button.set_sensitive (false);
+                       /* XXX we need to make sure that auto-play is off as well as insensitive */
+                       ActionManager::get_action ("Transport", "ToggleAutoPlay")->set_sensitive (false);
+                       ActionManager::get_action ("Transport", "ToggleAutoReturn")->set_sensitive (false);
+                       ActionManager::get_action ("Transport", "ToggleFollowEdits")->set_sensitive (false);
                }
-       }
-}
 
-void
-ARDOUR_UI::toggle_AutomaticallyCreateCrossfades()
-{
-}
+       } else if (p == "follow-edits") {
 
-void
-ARDOUR_UI::toggle_UnmuteNewFullCrossfades()
-{
-}
+               ActionManager::map_some_state ("Transport", "ToggleFollowEdits", &UIConfiguration::get_follow_edits);
 
-void
-ARDOUR_UI::mtc_port_changed ()
-{
-       bool have_mtc;
+       } else if (p == "send-mtc") {
+
+               ActionManager::map_some_state ("Options", "SendMTC", &RCConfiguration::get_send_mtc);
 
-       if (session) {
-               if (session->mtc_port()) {
-                       have_mtc = true;
+       } else if (p == "send-mmc") {
+
+               ActionManager::map_some_state ("Options", "SendMMC", &RCConfiguration::get_send_mmc);
+
+       } else if (p == "mmc-control") {
+               ActionManager::map_some_state ("Options", "UseMMC", &RCConfiguration::get_mmc_control);
+       } else if (p == "midi-feedback") {
+               ActionManager::map_some_state ("Options", "SendMIDIfeedback", &RCConfiguration::get_midi_feedback);
+       } else if (p == "auto-play") {
+               ActionManager::map_some_state ("Transport", "ToggleAutoPlay", sigc::mem_fun (_session->config, &SessionConfiguration::get_auto_play));
+       } else if (p == "auto-return") {
+               ActionManager::map_some_state ("Transport", "ToggleAutoReturn", sigc::mem_fun (_session->config, &SessionConfiguration::get_auto_return));
+       } else if (p == "auto-input") {
+               ActionManager::map_some_state ("Transport", "ToggleAutoInput", sigc::mem_fun (_session->config, &SessionConfiguration::get_auto_input));
+       } else if (p == "punch-out") {
+               ActionManager::map_some_state ("Transport", "TogglePunchOut", sigc::mem_fun (_session->config, &SessionConfiguration::get_punch_out));
+               if (!_session->config.get_punch_out()) {
+                       unset_dual_punch ();
+               }
+       } else if (p == "punch-in") {
+               ActionManager::map_some_state ("Transport", "TogglePunchIn", sigc::mem_fun (_session->config, &SessionConfiguration::get_punch_in));
+               if (!_session->config.get_punch_in()) {
+                       unset_dual_punch ();
+               }
+       } else if (p == "clicking") {
+               ActionManager::map_some_state ("Transport", "ToggleClick", &RCConfiguration::get_clicking);
+       } else if (p == "use-video-sync") {
+               ActionManager::map_some_state ("Transport",  "ToggleVideoSync", sigc::mem_fun (_session->config, &SessionConfiguration::get_use_video_sync));
+       } else if (p == "sync-source") {
+
+               synchronize_sync_source_and_video_pullup ();
+               set_fps_timeout_connection ();
+
+       } else if (p == "show-track-meters") {
+               if (editor) editor->toggle_meter_updating();
+       } else if (p == "primary-clock-delta-edit-cursor") {
+               if (UIConfiguration::instance().get_primary_clock_delta_edit_cursor()) {
+                       primary_clock->set_is_duration (true);
+                       primary_clock->set_editable (false);
+                       primary_clock->set_widget_name ("transport delta");
                } else {
-                       have_mtc = false;
+                       primary_clock->set_is_duration (false);
+                       primary_clock->set_editable (true);
+                       primary_clock->set_widget_name ("transport");
                }
-       } else {
-               have_mtc = false;
-       }
+       } else if (p == "secondary-clock-delta-edit-cursor") {
+               if (UIConfiguration::instance().get_secondary_clock_delta_edit_cursor()) {
+                       secondary_clock->set_is_duration (true);
+                       secondary_clock->set_editable (false);
+                       secondary_clock->set_widget_name ("secondary delta");
+               } else {
+                       secondary_clock->set_is_duration (false);
+                       secondary_clock->set_editable (true);
+                       secondary_clock->set_widget_name ("secondary");
+               }
+       } else if (p == "super-rapid-clock-update") {
+               if (_session) {
+                       stop_clocking ();
+                       start_clocking ();
+               }
+       } else if (p == "use-tooltips") {
+               /* this doesn't really belong here but it has to go somewhere */
+               if (UIConfiguration::instance().get_use_tooltips()) {
+                       Gtkmm2ext::enable_tooltips ();
+               } else {
+                       Gtkmm2ext::disable_tooltips ();
+               }
+       } else if (p == "waveform-gradient-depth") {
+               ArdourCanvas::WaveView::set_global_gradient_depth (UIConfiguration::instance().get_waveform_gradient_depth());
+       } else if (p == "show-editor-meter") {
+               bool show = UIConfiguration::instance().get_show_editor_meter();
+
+               if (editor_meter) {
+                       if (meter_box.get_parent()) {
+                               transport_hbox.remove (meter_box);
+                               transport_hbox.remove (editor_meter_peak_display);
+                       }
 
-       positional_sync_strings.clear ();
-       positional_sync_strings.push_back (slave_source_to_string (None));
-       if (have_mtc) {
-               positional_sync_strings.push_back (slave_source_to_string (MTC));
+                       if (show) {
+                               transport_hbox.pack_start (meter_box, false, false);
+                               transport_hbox.pack_start (editor_meter_peak_display, false, false);
+                               meter_box.show();
+                               editor_meter_peak_display.show();
+                       }
+               }
+       } else if (p == "waveform-scale") {
+               ArdourCanvas::WaveView::set_global_logscaled (UIConfiguration::instance().get_waveform_scale() == Logarithmic);
+       } else if (p == "widget-prelight") {
+               CairoWidget::set_widget_prelight (UIConfiguration::instance().get_widget_prelight());
+       } else if (p == "waveform-shape") {
+               ArdourCanvas::WaveView::set_global_shape (UIConfiguration::instance().get_waveform_shape() == Rectified
+                               ? ArdourCanvas::WaveView::Rectified : ArdourCanvas::WaveView::Normal);
+       } else if (p == "show-waveform-clipping") {
+               ArdourCanvas::WaveView::set_global_show_waveform_clipping (UIConfiguration::instance().get_show_waveform_clipping());
+       } else if (p == "waveform-cache-size") {
+               /* GUI option has units of megabytes; image cache uses units of bytes */
+               ArdourCanvas::WaveView::set_image_cache_size (UIConfiguration::instance().get_waveform_cache_size() * 1048576);
+       } else if (p == "use-wm-visibility") {
+               VisibilityTracker::set_use_window_manager_visibility (UIConfiguration::instance().get_use_wm_visibility());
+       } else if (p == "action-table-columns") {
+               const uint32_t cols = UIConfiguration::instance().get_action_table_columns ();
+               for (int i = 0; i < 9; ++i) {
+                       const int col = i / 3;
+                       if (cols & (1<<col)) {
+                               action_script_call_btn[i].show();
+                       } else {
+                               action_script_call_btn[i].hide();
+                       }
+               }
        }
-       positional_sync_strings.push_back (slave_source_to_string (JACK));
-       
-       set_popdown_strings (sync_option_combo, positional_sync_strings);
 }
 
 void
-ARDOUR_UI::setup_session_options ()
+ARDOUR_UI::session_parameter_changed (std::string p)
 {
-       mtc_port_changed ();
+       if (p == "native-file-data-format" || p == "native-file-header-format") {
+               update_format ();
+       } else if (p == "timecode-format") {
+               set_fps_timeout_connection ();
+       } else if (p == "video-pullup" || p == "timecode-format") {
+               set_fps_timeout_connection ();
 
-       Config->ParameterChanged.connect (mem_fun (*this, &ARDOUR_UI::parameter_changed));
+               synchronize_sync_source_and_video_pullup ();
+               reset_main_clocks ();
+               editor->queue_visual_videotimeline_update();
+       } else if (p == "track-name-number") {
+               /* DisplaySuspender triggers _route->redisplay() when going out of scope
+                * which eventually calls reset_controls_layout_width() and re-sets the
+                * track-header width.
+                * see also RouteTimeAxisView::update_track_number_visibility()
+                */
+               DisplaySuspender ds;
+       }
 }
 
 void
-ARDOUR_UI::map_some_state (const char* group, const char* action, bool (Configuration::*get)() const)
+ARDOUR_UI::reset_main_clocks ()
 {
-       Glib::RefPtr<Action> act = ActionManager::get_action (group, action);
-       if (act) {
-               Glib::RefPtr<ToggleAction> tact = Glib::RefPtr<ToggleAction>::cast_dynamic(act);
+       ENSURE_GUI_THREAD (*this, &ARDOUR_UI::reset_main_clocks)
 
-               if (tact) {
-                       bool x = (Config->*get)();
-                       
-                       if (tact->get_active() != x) {
-                               tact->set_active (x);
-                       }
-               }
+       if (_session) {
+               primary_clock->set (_session->audible_frame(), true);
+               secondary_clock->set (_session->audible_frame(), true);
+       } else {
+               primary_clock->set (0, true);
+               secondary_clock->set (0, true);
        }
 }
 
 void
-ARDOUR_UI::parameter_changed (const char* parameter_name)
+ARDOUR_UI::synchronize_sync_source_and_video_pullup ()
 {
-#define PARAM_IS(x) (!strcmp (parameter_name, (x)))
-
-       if (PARAM_IS ("slave-source")) {
-
-               sync_option_combo.set_active_text (slave_source_to_string (Config->get_slave_source()));
-
-       } else if (PARAM_IS ("send-mtc")) {
-
-               map_some_state ("options", "SendMTC", &Configuration::get_send_mtc);
-
-       } else if (PARAM_IS ("send-mmc")) {
-
-               map_some_state ("options", "SendMMC", &Configuration::get_send_mmc);
-
-       } else if (PARAM_IS ("mmc-control")) {
-               map_some_state ("options", "UseMMC", &Configuration::get_mmc_control);
-       } else if (PARAM_IS ("midi-feedback")) {
-               map_some_state ("options", "SendMIDIfeedback", &Configuration::get_midi_feedback);
-       } else if (PARAM_IS ("midi-control")) {
-               map_some_state ("options", "UseMIDIcontrol", &Configuration::get_midi_control);
-       } else if (PARAM_IS ("do-not-record-plugins")) {
-               map_some_state ("options", "DoNotRunPluginsWhileRecording", &Configuration::get_do_not_record_plugins);
-       } else if (PARAM_IS ("crossfades-active")) {
-               map_some_state ("options", "CrossfadesActive", &Configuration::get_crossfades_active);
-       } else if (PARAM_IS ("latched-record-enable")) {
-               map_some_state ("options", "LatchedRecordEnable", &Configuration::get_latched_record_enable);
-       } else if (PARAM_IS ("solo-latched")) {
-               map_some_state ("options", "LatchedSolo", &Configuration::get_solo_latched);
-       } else if (PARAM_IS ("solo-model")) {
-       } else if (PARAM_IS ("layer-model")) {
-       } else if (PARAM_IS ("crossfade-model")) {
-       } else if (PARAM_IS ("auto-play")) {
-               map_some_state ("Transport", "ToggleAutoPlay", &Configuration::get_auto_play);
-       } else if (PARAM_IS ("auto-loop")) {
-               map_some_state ("Transport", "Loop", &Configuration::get_auto_loop);
-       } else if (PARAM_IS ("auto-return")) {
-               map_some_state ("Transport", "ToggleAutoReturn", &Configuration::get_auto_return);
-       } else if (PARAM_IS ("auto-input")) {
-               map_some_state ("Transport", "ToggleAutoInput", &Configuration::get_auto_input);
-       } else if (PARAM_IS ("punch-out")) {
-               map_some_state ("Transport", "TogglePunchOut", &Configuration::get_punch_out);
-       } else if (PARAM_IS ("punch-in")) {
-               map_some_state ("Transport", "TogglePunchIn", &Configuration::get_punch_in);
-       } else if (PARAM_IS ("clicking")) {
-               map_some_state ("Transport", "ToggleClick", &Configuration::get_clicking);
-       } else if (PARAM_IS ("jack-time-master")) {
-               map_some_state ("Transport",  "ToggleTimeMaster", &Configuration::get_jack_time_master);
-       } else if (PARAM_IS ("plugins-stop-with-transport")) {
-               map_some_state ("options",  "StopPluginsWithTransport", &Configuration::get_plugins_stop_with_transport);
-       } else if (PARAM_IS ("latched-record-enable")) {
-               map_some_state ("options", "LatchedRecordEnable", &Configuration::get_latched_record_enable);
-       } else if (PARAM_IS ("verify-remove-last-capture")) {
-               map_some_state ("options",  "VerifyRemoveLastCapture", &Configuration::get_verify_remove_last_capture);
-       } else if (PARAM_IS ("stop-recording-on-xrun")) {
-               map_some_state ("options",  "StopRecordingOnXrun", &Configuration::get_stop_recording_on_xrun);
-       } else if (PARAM_IS ("stop-at-session-end")) {
-               map_some_state ("options",  "StopTransportAtEndOfSession", &Configuration::get_stop_at_session_end);
-       } else if (PARAM_IS ("use-hardware-monitoring")) {
-               map_some_state ("options",  "UseHardwareMonitoring", &Configuration::get_use_hardware_monitoring);
-       } else if (PARAM_IS ("use-sw-monitoring")) {
-               map_some_state ("options",  "UseSoftwareMonitoring", &Configuration::get_use_sw_monitoring);
-       } else if (PARAM_IS ("use-external-monitoring")) {
-               map_some_state ("options",  "UseExternalMonitoring", &Configuration::get_use_external_monitoring);
-       } else if (PARAM_IS ("use-video-sync")) {
-               map_some_state ("Transport",  "ToggleVideoSync", &Configuration::get_use_video_sync);
-       } else if (PARAM_IS ("quieten-at-speed")) {
-               map_some_state ("options",  "GainReduceFastTransport", &Configuration::get_quieten_at_speed);
-       } else if (PARAM_IS ("shuttle-behaviour")) {
-
-               switch (Config->get_shuttle_behaviour ()) {
-               case Sprung:
-                       shuttle_style_button.set_active_text (_("sprung"));
-                       shuttle_fract = 0.0;
-                       shuttle_box.queue_draw ();
-                       if (session) {
-                               if (session->transport_rolling()) {
-                                       shuttle_fract = SHUTTLE_FRACT_SPEED1;
-                                       session->request_transport_speed (1.0);
-                               }
-                       }
-                       break;
-               case Wheel:
-                       shuttle_style_button.set_active_text (_("wheel"));
-                       break;
-               }
+       Glib::RefPtr<Action> act = ActionManager::get_action (X_("Transport"), X_("ToggleExternalSync"));
 
-       } else if (PARAM_IS ("shuttle-units")) {
-               
-               switch (Config->get_shuttle_units()) {
-               case Percentage:
-                       shuttle_units_button.set_label("% ");
-                       break;
-               case Semitones:
-                       shuttle_units_button.set_label(_("ST"));
-                       break;
+       if (!act) {
+               return;
+       }
+
+       if (!_session) {
+               goto just_label;
+       }
+
+       if (_session->config.get_video_pullup() == 0.0f) {
+               /* with no video pull up/down, any sync source is OK */
+               act->set_sensitive (true);
+       } else {
+               /* can't sync to JACK if video pullup != 0.0 */
+               if (Config->get_sync_source() == Engine) {
+                       act->set_sensitive (false);
+               } else {
+                       act->set_sensitive (true);
                }
        }
-       
-#undef PARAM_IS
+
+       /* XXX should really be able to set the video pull up
+          action to insensitive/sensitive, but there is no action.
+          FIXME
+       */
+
+  just_label:
+       if (act->get_sensitive ()) {
+               set_tip (sync_button, _("Enable/Disable external positional sync"));
+       } else {
+               set_tip (sync_button, _("Sync to JACK is not possible: video pull up/down is set"));
+       }
+
 }
+