Update codec-quality display when ExportFormat changes
[ardour.git] / gtk2_ardour / rc_option_editor.cc
index 0f33de33ce46178cb06bedd296f95a7bfefff8f6..c1a9b87527c549d058f984e88bfdf167822ef958 100644 (file)
 
 #include "ardour/audio_backend.h"
 #include "ardour/audioengine.h"
-#include "ardour/profile.h"
-#include "ardour/dB.h"
-#include "ardour/rc_configuration.h"
 #include "ardour/control_protocol_manager.h"
+#include "ardour/dB.h"
 #include "ardour/port_manager.h"
 #include "ardour/plugin_manager.h"
+#include "ardour/profile.h"
+#include "ardour/rc_configuration.h"
+#include "ardour/transport_master_manager.h"
+
 #include "control_protocol/control_protocol.h"
 
 #include "waveview/wave_view.h"
@@ -66,6 +68,7 @@
 #include "midi_tracer.h"
 #include "rc_option_editor.h"
 #include "sfdb_ui.h"
+#include "transport_masters_dialog.h"
 #include "ui_config.h"
 #include "utils.h"
 
@@ -136,7 +139,7 @@ public:
                if (_rc_config->get_click_sound ().empty() &&
                    _rc_config->get_click_emphasis_sound().empty()) {
                        _use_default_click_check_button.set_active (true);
-                       _use_emphasis_on_click_check_button.set_active (true);
+                       _use_emphasis_on_click_check_button.set_active (_rc_config->get_use_click_emphasis ());
 
                } else {
                        _use_default_click_check_button.set_active (false);
@@ -1404,10 +1407,12 @@ class ControlSurfacesOptions : public OptionEditorMiniPage
                {
                        //enable the Edit button when a row is selected for editing
                        TreeModel::Row row = *(_view.get_selection()->get_selected());
-                       if (row && row[_model.enabled])
-                               edit_button->set_sensitive (true);
-                       else
+                       if (row && row[_model.enabled]) {
+                               ControlProtocolInfo* cpi = row[_model.protocol_info];
+                               edit_button->set_sensitive (cpi && cpi->protocol && cpi->protocol->has_editor ());
+                       } else {
                                edit_button->set_sensitive (false);
+                       }
                }
 
                void view_changed (TreeModel::Path const &, TreeModel::iterator const & i)
@@ -1589,9 +1594,6 @@ class VideoTimelineOptions : public OptionEditorMiniPage
                        _video_server_docroot_entry.signal_activate().connect (sigc::mem_fun(*this, &VideoTimelineOptions::server_docroot_changed));
                        _custom_xjadeo_path.signal_changed().connect (sigc::mem_fun (*this, &VideoTimelineOptions::custom_xjadeo_path_changed));
                        _xjadeo_browse_button.signal_clicked ().connect (sigc::mem_fun (*this, &VideoTimelineOptions::xjadeo_browse_clicked));
-
-                       // xjadeo-path is a UIConfig parameter
-                       UIConfiguration::instance().ParameterChanged.connect (sigc::mem_fun (*this, &VideoTimelineOptions::parameter_changed));
                }
 
                void server_url_changed ()
@@ -1624,13 +1626,13 @@ class VideoTimelineOptions : public OptionEditorMiniPage
 
                void custom_xjadeo_path_changed ()
                {
-                       UIConfiguration::instance().set_xjadeo_binary (_custom_xjadeo_path.get_text());
+                       _rc_config->set_xjadeo_binary (_custom_xjadeo_path.get_text());
                }
 
                void xjadeo_browse_clicked ()
                {
                        Gtk::FileChooserDialog dialog(_("Set Video Monitor Executable"), Gtk::FILE_CHOOSER_ACTION_OPEN);
-                       dialog.set_filename (UIConfiguration::instance().get_xjadeo_binary());
+                       dialog.set_filename (_rc_config->get_xjadeo_binary());
                        dialog.add_button(Gtk::Stock::CANCEL, Gtk::RESPONSE_CANCEL);
                        dialog.add_button(Gtk::Stock::OK, Gtk::RESPONSE_OK);
                        if (dialog.run () == Gtk::RESPONSE_OK) {
@@ -1641,7 +1643,7 @@ class VideoTimelineOptions : public OptionEditorMiniPage
 #endif
                                                        Glib::file_test (filename, Glib::FILE_TEST_EXISTS|Glib::FILE_TEST_IS_EXECUTABLE)
                                                        )) {
-                                       UIConfiguration::instance().set_xjadeo_binary (filename);
+                                       _rc_config->set_xjadeo_binary (filename);
                                }
                        }
                }
@@ -1664,7 +1666,7 @@ class VideoTimelineOptions : public OptionEditorMiniPage
                                _video_server_docroot_entry.set_sensitive(x);
                                _video_server_url_entry.set_sensitive(x);
                        } else if (p == "xjadeo-binary") {
-                               _custom_xjadeo_path.set_text (UIConfiguration::instance().get_xjadeo_binary());
+                               _custom_xjadeo_path.set_text (_rc_config->get_xjadeo_binary());
                        }
                }
 
@@ -2107,14 +2109,7 @@ MidiPortOptions::pretty_name_edit (std::string const & path, string const & new_
                return;
        }
 
-       boost::shared_ptr<ARDOUR::AudioBackend> backend = ARDOUR::AudioEngine::instance()->current_backend();
-       if (backend) {
-               ARDOUR::PortEngine::PortHandle ph = backend->get_port_by_name ((*iter)[midi_port_columns.name]);
-               if (ph) {
-                       backend->set_port_property (ph, "http://jackaudio.org/metadata/pretty-name", new_text, "");
-                       (*iter)[midi_port_columns.pretty_name] = new_text;
-               }
-       }
+       AudioEngine::instance()->set_midi_port_pretty_name ((*iter)[midi_port_columns.name], new_text);
 }
 
 /*============*/
@@ -2136,7 +2131,6 @@ RCOptionEditor::RCOptionEditor ()
 
        uint32_t hwcpus = hardware_concurrency ();
        BoolOption* bo;
-       BoolComboOption* bco;
 
        if (hwcpus > 1) {
                add_option (_("General"), new OptionEditorHeading (_("DSP CPU Utilization")));
@@ -2283,7 +2277,7 @@ RCOptionEditor::RCOptionEditor ()
                     ));
 
 
-#ifdef ENABLE_NLS
+#if ENABLE_NLS
 
        add_option (_("General/Translation"), new OptionEditorHeading (_("Internationalization")));
 
@@ -2306,14 +2300,6 @@ RCOptionEditor::RCOptionEditor ()
 
        add_option (_("Editor"), new OptionEditorHeading (_("General")));
 
-       add_option (_("Editor"),
-            new BoolOption (
-                    "rubberbanding-snaps-to-grid",
-                    _("Snap rubberband to grid"),
-                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::get_rubberbanding_snaps_to_grid),
-                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::set_rubberbanding_snaps_to_grid)
-                    ));
-
        bo = new BoolOption (
                     "name-new-markers",
                     _("Prompt for new marker names"),
@@ -2450,16 +2436,18 @@ RCOptionEditor::RCOptionEditor ()
 
        add_option (_("Editor"), fadeshape);
 
-       bco = new BoolComboOption (
-                    "use-overlap-equivalency",
-                    _("Regions in edit groups are edited together"),
-                    _("whenever they overlap in time"),
-                    _("only if they have identical length and position"),
-                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_use_overlap_equivalency),
-                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_use_overlap_equivalency)
+       ComboOption<RegionEquivalence> *eqv = new ComboOption<RegionEquivalence> (
+                    "region-equivalence",
+                    _("Regions in active edit groups are edited together"),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_region_equivalence),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_region_equivalence)
                     );
 
-       add_option (_("Editor"), bco);
+       eqv->add (Overlap, _("whenever they overlap in time"));
+       eqv->add (Enclosed, _("if either encloses the other"));
+       eqv->add (Exact, _("only if they have identical length, position and origin"));
+
+       add_option (_("Editor"), eqv);
 
        ComboOption<LayerModel>* lm = new ComboOption<LayerModel> (
                "layer-model",
@@ -2490,6 +2478,93 @@ RCOptionEditor::RCOptionEditor ()
 
        add_option (_("Editor"), rsas);
 
+       add_option (_("Editor/Snap"), new OptionEditorHeading (_("General Snap options:")));
+
+       add_option (_("Editor/Snap"),
+                   new SpinOption<uint32_t> (
+                           "snap-threshold",
+                           _("Snap Threshold (pixels)"),
+                           sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::get_snap_threshold),
+                           sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::set_snap_threshold),
+                           10, 200,
+                           1, 10
+                           ));
+
+       add_option (_("Editor/Snap"),
+            new BoolOption (
+                    "show-snapped-cursor",
+                    _("Show \"snapped cursor\""),
+                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::get_show_snapped_cursor),
+                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::set_show_snapped_cursor)
+                    ));
+
+       add_option (_("Editor/Snap"),
+            new BoolOption (
+                    "rubberbanding-snaps-to-grid",
+                    _("Snap rubberband selection to grid"),
+                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::get_rubberbanding_snaps_to_grid),
+                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::set_rubberbanding_snaps_to_grid)
+                    ));
+
+       add_option (_("Editor/Snap"),
+            new BoolOption (
+                    "grid-follows-internal",
+                    _("Grid switches to alternate selection for Internal Edit tools"),
+                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::get_grid_follows_internal),
+                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::set_grid_follows_internal)
+                    ));
+
+       add_option (_("Editor/Snap"),
+            new BoolOption (
+                    "rulers-follow-grid",
+                    _("Rulers automatically change to follow the Grid mode selection"),
+                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::get_rulers_follow_grid),
+                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::set_rulers_follow_grid)
+                    ));
+
+       add_option (_("Editor/Snap"), new OptionEditorHeading (_("When \"Snap\" is enabled, snap to:")));
+
+
+       add_option (_("Editor/Snap"),
+            new BoolOption (
+                    "snap-to-marks",
+                    _("Markers"),
+                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::get_snap_to_marks),
+                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::set_snap_to_marks)
+                    ));
+
+       add_option (_("Editor/Snap"),
+            new BoolOption (
+                    "snap-to-region-sync",
+                    _("Region Sync Points"),
+                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::get_snap_to_region_sync),
+                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::set_snap_to_region_sync)
+                    ));
+
+       add_option (_("Editor/Snap"),
+            new BoolOption (
+                    "snap-to-region-start",
+                    _("Region Starts"),
+                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::get_snap_to_region_start),
+                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::set_snap_to_region_start)
+                    ));
+
+       add_option (_("Editor/Snap"),
+            new BoolOption (
+                    "snap-to-region-end",
+                    _("Region Ends"),
+                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::get_snap_to_region_end),
+                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::set_snap_to_region_end)
+                    ));
+
+       add_option (_("Editor/Snap"),
+            new BoolOption (
+                    "snap-to-grid",
+                    _("Grid"),
+                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::get_snap_to_grid),
+                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::set_snap_to_grid)
+                    ));
+
        add_option (_("Editor/Modifiers"), new OptionEditorHeading (_("Keyboard Modifiers")));
        add_option (_("Editor/Modifiers"), new KeyboardOptions);
        add_option (_("Editor/Modifiers"), new OptionEditorBlank ());
@@ -3131,16 +3206,9 @@ RCOptionEditor::RCOptionEditor ()
 
        /* SYNC */
 
-       add_option (_("Sync"), new OptionEditorHeading (_("External Synchronization")));
+       add_option (_("Sync"), new OptionEditorHeading (_("Transport Masters")));
 
-       _sync_source = new ComboOption<SyncSource> (
-               "sync-source",
-               _("External timecode source"),
-               sigc::mem_fun (*_rc_config, &RCConfiguration::get_sync_source),
-               sigc::mem_fun (*_rc_config, &RCConfiguration::set_sync_source)
-               );
-
-       add_option (_("Sync"), _sync_source);
+       add_option (_("Sync"), new WidgetOption (X_("foo"), X_("Transport Masters"), _transport_masters_widget));
 
        _sync_framerate = new BoolOption (
                     "timecode-sync-frame-rate",
@@ -3158,64 +3226,6 @@ RCOptionEditor::RCOptionEditor ()
 
        add_option (_("Sync"), _sync_framerate);
 
-       _sync_genlock = new BoolOption (
-               "timecode-source-is-synced",
-               _("Sync-lock timecode to clock (disable drift compensation)"),
-               sigc::mem_fun (*_rc_config, &RCConfiguration::get_timecode_source_is_synced),
-               sigc::mem_fun (*_rc_config, &RCConfiguration::set_timecode_source_is_synced)
-               );
-       Gtkmm2ext::UI::instance()->set_tip
-               (_sync_genlock->tip_widget(),
-                string_compose (_("<b>When enabled</b> %1 will never varispeed when slaved to external timecode. "
-                                  "Sync Lock indicates that the selected external timecode source shares clock-sync "
-                                  "(Black &amp; Burst, Wordclock, etc) with the audio interface. "
-                                  "This option disables drift compensation. The transport speed is fixed at 1.0. "
-                                  "Vari-speed LTC will be ignored and cause drift."
-                                  "\n\n"
-                                  "<b>When disabled</b> %1 will compensate for potential drift, regardless if the "
-                                  "timecode sources shares clock sync."
-                                 ), PROGRAM_NAME));
-
-
-       add_option (_("Sync"), _sync_genlock);
-
-       _sync_source_2997 = new BoolOption (
-               "timecode-source-2997",
-               _("Lock to 29.9700 fps instead of 30000/1001"),
-               sigc::mem_fun (*_rc_config, &RCConfiguration::get_timecode_source_2997),
-               sigc::mem_fun (*_rc_config, &RCConfiguration::set_timecode_source_2997)
-               );
-       Gtkmm2ext::UI::instance()->set_tip
-               (_sync_source_2997->tip_widget(),
-                _("<b>When enabled</b> the external timecode source is assumed to use 29.97 fps instead of 30000/1001.\n"
-                        "SMPTE 12M-1999 specifies 29.97df as 30000/1001. The spec further mentions that "
-                        "drop-sample timecode has an accumulated error of -86ms over a 24-hour period.\n"
-                        "Drop-sample timecode would compensate exactly for a NTSC color frame rate of 30 * 0.9990 (ie 29.970000). "
-                        "That is not the actual rate. However, some vendors use that rate - despite it being against the specs - "
-                        "because the variant of using exactly 29.97 fps has zero timecode drift.\n"
-                        ));
-
-       add_option (_("Sync"), _sync_source_2997);
-
-       add_option (_("Sync/LTC"), new OptionEditorHeading (_("Linear Timecode (LTC) Reader")));
-
-       _ltc_port = new ComboStringOption (
-               "ltc-source-port",
-               _("LTC incoming port"),
-               sigc::mem_fun (*_rc_config, &RCConfiguration::get_ltc_source_port),
-               sigc::mem_fun (*_rc_config, &RCConfiguration::set_ltc_source_port)
-               );
-
-       vector<string> physical_inputs;
-       physical_inputs.push_back (_("None"));
-       AudioEngine::instance()->get_physical_inputs (DataType::AUDIO, physical_inputs);
-       _ltc_port->set_popdown_strings (physical_inputs);
-
-       populate_sync_options ();
-       AudioEngine::instance()->Running.connect (engine_started_connection, MISSING_INVALIDATOR, boost::bind (&RCOptionEditor::populate_sync_options, this), gui_context());
-
-       add_option (_("Sync/LTC"), _ltc_port);
-
        add_option (_("Sync/LTC"), new OptionEditorHeading (_("Linear Timecode (LTC) Generator")));
 
        add_option (_("Sync/LTC"),
@@ -3303,7 +3313,7 @@ RCOptionEditor::RCOptionEditor ()
                     _("Inbound MMC device ID"),
                     sigc::mem_fun (*_rc_config, &RCConfiguration::get_mmc_receive_device_id),
                     sigc::mem_fun (*_rc_config, &RCConfiguration::set_mmc_receive_device_id),
-                    0, 128, 1, 10
+                    0, 127, 1, 10
                     ));
 
        add_option (_("Sync/MIDI"),
@@ -3312,7 +3322,7 @@ RCOptionEditor::RCOptionEditor ()
                     _("Outbound MMC device ID"),
                     sigc::mem_fun (*_rc_config, &RCConfiguration::get_mmc_send_device_id),
                     sigc::mem_fun (*_rc_config, &RCConfiguration::set_mmc_send_device_id),
-                    0, 128, 1, 10
+                    0, 127, 1, 10
                     ));
 
 
@@ -3326,7 +3336,7 @@ RCOptionEditor::RCOptionEditor ()
 
        add_option (_("MIDI Ports"),
                    new BoolOption (
-                           "get-midi-input-follows-selection",
+                           "midi-input-follows-selection",
                            _("MIDI input follows MIDI track selection"),
                            sigc::mem_fun (*_rc_config, &RCConfiguration::get_midi_input_follows_selection),
                            sigc::mem_fun (*_rc_config, &RCConfiguration::set_midi_input_follows_selection)
@@ -3379,6 +3389,23 @@ RCOptionEditor::RCOptionEditor ()
        Gtkmm2ext::UI::instance()->set_tip (bo->tip_widget(),
                                            _("<b>When enabled</b> plugins will be activated when they are added to tracks/busses. When disabled plugins will be left inactive when they are added to tracks/busses"));
 
+       ComboOption<uint32_t>* lna = new ComboOption<uint32_t> (
+                    "limit-n-automatables",
+                    _("Limit automatable parameters per plugin"),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_limit_n_automatables),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_limit_n_automatables)
+                    );
+       lna->add (0, _("Unlimited"));
+       lna->add (64,  _("64 parameters"));
+       lna->add (128, _("128 parameters"));
+       lna->add (256, _("256 parameters"));
+       lna->add (512, _("512 parameters"));
+       lna->add (999, _("999 parameters"));
+       add_option (_("Plugins"), lna);
+       Gtkmm2ext::UI::instance()->set_tip (lna->tip_widget(),
+                                           _("Some Plugins expose an unreasonable amount of control-inputs. This option limits the number of parameters that can are listed as automatable without restricting the number of total controls.\n\nThis reduces lag in the GUI and shortens excessively long drop-down lists for plugins with a large number of control ports.\n\nNote: This only affects newly added plugins and is applied to plugin on session-reload. Already automated parameters are retained."));
+
+
 #if (defined WINDOWS_VST_SUPPORT || defined MACVST_SUPPORT || defined LXVST_SUPPORT)
        add_option (_("Plugins/VST"), new OptionEditorHeading (_("VST")));
 #if 0
@@ -3728,6 +3755,23 @@ RCOptionEditor::RCOptionEditor ()
                     sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::set_never_display_periodic_midi)
                     ));
 
+
+       add_option (_("Appearance/Editor"),
+                   new BoolOption (
+                           "use-note-bars-for-velocity",
+                           _("Show velocity horizontally inside notes"),
+                           sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::get_use_note_bars_for_velocity),
+                           sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::set_use_note_bars_for_velocity)
+                           ));
+
+       add_option (_("Appearance/Editor"),
+                   new BoolOption (
+                           "use-note-color-for-velocity",
+                           _("Use colors to show note velocity"),
+                           sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::get_use_note_color_for_velocity),
+                           sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::set_use_note_color_for_velocity)
+                           ));
+
        add_option (_("Appearance/Editor"), new OptionEditorBlank ());
 
        /* The names of these controls must be the same as those given in MixerStrip
@@ -3758,6 +3802,19 @@ RCOptionEditor::RCOptionEditor ()
                     sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::set_default_narrow_ms)
                     ));
 
+       ComboOption<uint32_t>* mic = new ComboOption<uint32_t> (
+                    "max-inline-controls",
+                    _("Limit inline-mixer-strip controls per plugin"),
+                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::get_max_inline_controls),
+                    sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::set_max_inline_controls)
+                    );
+       mic->add (0, _("Unlimited"));
+       mic->add (16,  _("16 parameters"));
+       mic->add (32,  _("32 parameters"));
+       mic->add (64,  _("64 parameters"));
+       mic->add (128, _("128 parameters"));
+       add_option (_("Appearance/Mixer"), mic);
+
        add_option (_("Appearance/Mixer"), new OptionEditorBlank ());
 
        add_option (_("Appearance/Toolbar"), new OptionEditorHeading (_("Main Transport Toolbar Items")));
@@ -3814,7 +3871,7 @@ RCOptionEditor::RCOptionEditor ()
 
        add_option (_("Appearance/Toolbar"),
                        new ColumVisibilityOption (
-                               "action-table-columns", _("Lua Action Script Button Visibility"), 4,
+                               "action-table-columns", _("Display Action-Buttons"), 4,
                                sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::get_action_table_columns),
                                sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::set_action_table_columns)
                                )
@@ -3833,6 +3890,13 @@ RCOptionEditor::RCOptionEditor ()
                                sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::set_flat_buttons)
                                ));
 
+       add_option (_("Appearance/Theme"), new BoolOption (
+                               "boxy-buttons",
+                               _("Draw \"boxy\" buttons"),
+                               sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::get_boxy_buttons),
+                               sigc::mem_fun (UIConfiguration::instance(), &UIConfiguration::set_boxy_buttons)
+                               ));
+
        add_option (_("Appearance/Theme"), new BoolOption (
                                "meter-style-led",
                                _("LED meter style"),
@@ -3959,6 +4023,13 @@ These settings will only take effect after %1 is restarted.\n\
        set_current_page (_("General"));
 }
 
+void
+RCOptionEditor::set_session (Session *s)
+{
+       SessionHandlePtr::set_session (s);
+       _transport_masters_widget.set_session (s);
+}
+
 void
 RCOptionEditor::parameter_changed (string const & p)
 {
@@ -3973,22 +4044,11 @@ RCOptionEditor::parameter_changed (string const & p)
                _solo_control_is_listen_control->set_sensitive (s);
                _listen_position->set_sensitive (s);
        } else if (p == "sync-source") {
-               _sync_source->set_sensitive (true);
-               if (_session) {
-                       _sync_source->set_sensitive (!_session->config.get_external_sync());
-               }
-               switch(Config->get_sync_source()) {
-               case ARDOUR::MTC:
-               case ARDOUR::LTC:
-                       _sync_genlock->set_sensitive (true);
+               boost::shared_ptr<TransportMaster> tm (TransportMasterManager::instance().current());
+               if (boost::dynamic_pointer_cast<TimecodeTransportMaster> (tm)) {
                        _sync_framerate->set_sensitive (true);
-                       _sync_source_2997->set_sensitive (true);
-                       break;
-               default:
-                       _sync_genlock->set_sensitive (false);
+               } else {
                        _sync_framerate->set_sensitive (false);
-                       _sync_source_2997->set_sensitive (false);
-                       break;
                }
        } else if (p == "send-ltc") {
                bool const s = Config->get_send_ltc ();
@@ -4065,29 +4125,6 @@ void RCOptionEditor::edit_vst_path () {
        delete pd;
 }
 
-
-void
-RCOptionEditor::populate_sync_options ()
-{
-       vector<SyncSource> sync_opts = ARDOUR::get_available_sync_options ();
-
-       _sync_source->clear ();
-
-       for (vector<SyncSource>::iterator i = sync_opts.begin(); i != sync_opts.end(); ++i) {
-               _sync_source->add (*i, sync_source_to_string (*i));
-       }
-
-       if (sync_opts.empty()) {
-               _sync_source->set_sensitive(false);
-       } else {
-               if (std::find(sync_opts.begin(), sync_opts.end(), _rc_config->get_sync_source()) == sync_opts.end()) {
-                       _rc_config->set_sync_source(sync_opts.front());
-               }
-       }
-
-       parameter_changed ("sync-source");
-}
-
 Gtk::Window*
 RCOptionEditor::use_own_window (bool and_fill_it)
 {