diff --git a/gtk2_ardour/ardour_ui_ed.cc b/gtk2_ardour/ardour_ui_ed.cc index c89a617a24..cf31dbb9f3 100644 --- a/gtk2_ardour/ardour_ui_ed.cc +++ b/gtk2_ardour/ardour_ui_ed.cc @@ -121,6 +121,7 @@ ARDOUR_UI::populate_display_format_dropdown () { static std::vector display_formats; _display_format_dropdown->clear_items (); + display_formats.clear (); display_formats.push_back("Timecode"); display_formats.push_back("Time"); @@ -134,45 +135,38 @@ ARDOUR_UI::populate_display_format_dropdown () if( !_session ) return; - string format = _session->config.get_display_format(); - AudioClock::Mode mode; - - if ( format == "Time" ) - mode = AudioClock::MinSec; - else if ( format == "Samples" ) - mode = AudioClock::Frames; - if( format == "Timecode" ) - mode = AudioClock::Timecode; - - if( time_info_box ) - time_info_box->set_mode (mode); - if ( big_clock ) - primary_clock->set_mode (mode); - + AudioClock::Mode mode = primary_clock->mode(); + + string format; + if (AudioClock::Timecode == mode) + format = display_formats[0]; + else if (AudioClock::MinSec == mode) + format = display_formats[1]; + else format = display_formats[2]; + _display_format_dropdown->set_text( format ); } void ARDOUR_UI::populate_timecode_source_dropdown () { - static std::vector timecode_selector; + static std::vector timecode_source; _timecode_source_dropdown->clear_items (); - timecode_selector.push_back("Internal"); - timecode_selector.push_back("MTC"); - timecode_selector.push_back("LTC"); - - for(int i = 0; i < timecode_selector.size(); ++i) + timecode_source.clear(); + timecode_source.push_back("Internal"); + timecode_source.push_back("MTC"); + timecode_source.push_back("LTC"); + + for(int i = 0; i < timecode_source.size(); ++i) { - _timecode_source_dropdown->add_menu_item (timecode_selector[i], &timecode_selector[i]); + _timecode_source_dropdown->add_menu_item (timecode_source[i], &timecode_source[i]); } if( !_session ) return; - string timecode_source = _session->config.get_timecode_source(); - - _timecode_source_dropdown->set_text( timecode_source ); + _timecode_source_dropdown->set_text (timecode_source[0]); } void @@ -187,8 +181,6 @@ ARDOUR_UI::on_display_format_dropdown_item_clicked (WavesDropdown* from_which, v mode = AudioClock::MinSec; else if ( format == "Samples" ) mode = AudioClock::Frames; - - _session->config.set_display_format(format); if( time_info_box ) time_info_box->set_mode (mode); @@ -200,7 +192,6 @@ void ARDOUR_UI::on_timecode_source_dropdown_item_clicked (WavesDropdown* from_which, void* my_cookie) { string timecode_source = *((string*)my_cookie); - _session->config.set_timecode_source(timecode_source); } void diff --git a/libs/ardour/ardour/session_configuration_vars.h b/libs/ardour/ardour/session_configuration_vars.h index 56ccb61fa4..8739e69c86 100644 --- a/libs/ardour/ardour/session_configuration_vars.h +++ b/libs/ardour/ardour/session_configuration_vars.h @@ -30,8 +30,6 @@ CONFIG_VARIABLE (bool, use_region_fades, "use-region-fades", true) CONFIG_VARIABLE (bool, show_region_fades, "show-region-fades", true) CONFIG_VARIABLE (SampleFormat, native_file_data_format, "native-file-data-format", ARDOUR::FormatFloat) CONFIG_VARIABLE (HeaderFormat, native_file_header_format, "native-file-header-format", ARDOUR::WAVE) -CONFIG_VARIABLE (std::string, display_format, "display-format", std::string("Timecode")) -CONFIG_VARIABLE (std::string, timecode_source, "timecode-source", std::string("Internal")) CONFIG_VARIABLE (bool, auto_play, "auto-play", false) CONFIG_VARIABLE (bool, auto_return, "auto-return", false) CONFIG_VARIABLE (bool, auto_input, "auto-input", true)