X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Fengine_dialog.cc;h=69dc179798b71c515494640b1353cdf4d8609365;hb=948c5d07065e862704da2d5f02207ed87ca821aa;hp=a89ddddb1a9ec7e5ff295ef4c0055bd96922f446;hpb=33ea2d25f1e11bf97f36053eaa367c2e121ae8fe;p=ardour.git diff --git a/gtk2_ardour/engine_dialog.cc b/gtk2_ardour/engine_dialog.cc index a89ddddb1a..69dc179798 100644 --- a/gtk2_ardour/engine_dialog.cc +++ b/gtk2_ardour/engine_dialog.cc @@ -20,7 +20,6 @@ #include #include #include -#include #include #include @@ -28,6 +27,7 @@ #include #include "pbd/error.h" +#include "pbd/locale_guard.h" #include "pbd/xml++.h" #include "pbd/unwind.h" #include "pbd/failed_constructor.h" @@ -43,21 +43,31 @@ #include "ardour/mididm.h" #include "ardour/rc_configuration.h" #include "ardour/types.h" +#include "ardour/profile.h" #include "pbd/convert.h" #include "pbd/error.h" +#include "opts.h" +#include "debug.h" #include "ardour_ui.h" #include "engine_dialog.h" #include "gui_thread.h" +#include "ui_config.h" +#include "public_editor.h" #include "utils.h" -#include "i18n.h" +#include "pbd/i18n.h" +#include "splash.h" using namespace std; using namespace Gtk; using namespace Gtkmm2ext; using namespace PBD; using namespace Glib; +using namespace ArdourWidgets; +using namespace ARDOUR_UI_UTILS; + +#define DEBUG_ECONTROL(msg) DEBUG_TRACE (PBD::DEBUG::EngineControl, string_compose ("%1: %2\n", __LINE__, msg)); static const unsigned int midi_tab = 2; static const unsigned int latency_tab = 1; /* zero-based, page zero is the main setup page */ @@ -66,6 +76,7 @@ static const char* results_markup = X_("%1 EngineControl::EngineControl () : ArdourDialog (_("Audio/MIDI Setup")) + , engine_status ("") , basic_packer (9, 4) , input_latency_adjustment (0, 0, 99999, 1) , input_latency (input_latency_adjustment) @@ -79,6 +90,9 @@ EngineControl::EngineControl () , ports_spinner (ports_adjustment) , control_app_button (_("Device Control Panel")) , midi_devices_button (_("Midi Device Setup")) + , start_stop_button (_("Stop")) + , update_devices_button (_("Refresh Devices")) + , use_buffered_io_button (_("Use Buffered I/O"), ArdourButton::led_default_elements) , lm_measure_label (_("Measure")) , lm_use_button (_("Use results")) , lm_back_button (_("Back to settings ... (ignore results)")) @@ -88,11 +102,15 @@ EngineControl::EngineControl () , lm_running (false) , midi_back_button (_("Back to settings")) , ignore_changes (0) + , ignore_device_changes (0) , _desired_sample_rate (0) , started_at_least_once (false) + , queue_device_changed (false) + , _have_control (true) + , block_signals(0) { using namespace Notebook_Helpers; - vector strings; + vector backend_names; Label* label; AttachOptions xopt = AttachOptions (FILL|EXPAND); int row; @@ -110,12 +128,10 @@ EngineControl::EngineControl () } for (vector::const_iterator b = backends.begin(); b != backends.end(); ++b) { - strings.push_back ((*b)->name); + backend_names.push_back ((*b)->name); } - set_popdown_strings (backend_combo, strings); - backend_combo.set_active_text (strings.front()); - backend_combo.signal_changed().connect (sigc::mem_fun (*this, &EngineControl::backend_changed)); + set_popdown_strings (backend_combo, backend_names); /* setup basic packing characteristics for the table used on the main * tab of the notebook @@ -173,13 +189,12 @@ EngineControl::EngineControl () lm_table.attach (*misc_align, 1, 3, row, row+1, FILL, (AttachOptions) 0); ++row; - xopt = AttachOptions(0); - lm_measure_label.set_padding (10, 10); lm_measure_button.add (lm_measure_label); lm_measure_button.signal_clicked().connect (sigc::mem_fun (*this, &EngineControl::latency_button_clicked)); lm_use_button.signal_clicked().connect (sigc::mem_fun (*this, &EngineControl::use_latency_button_clicked)); - lm_back_button_signal = lm_back_button.signal_clicked().connect (sigc::bind (sigc::mem_fun (notebook, &Gtk::Notebook::set_current_page), 0)); + lm_back_button_signal = lm_back_button.signal_clicked().connect( + sigc::mem_fun(*this, &EngineControl::latency_back_button_clicked)); lm_use_button.set_sensitive (false); @@ -254,16 +269,30 @@ EngineControl::EngineControl () output_channels.signal_output().connect (sigc::bind (sigc::ptr_fun (&EngineControl::print_channel_count), &output_channels)); midi_devices_button.signal_clicked.connect (mem_fun (*this, &EngineControl::configure_midi_devices)); - midi_devices_button.set_sensitive (false); midi_devices_button.set_name ("generic button"); midi_devices_button.set_can_focus(true); - control_app_button.signal_clicked().connect (mem_fun (*this, &EngineControl::control_app_button_clicked)); + control_app_button.signal_clicked.connect (mem_fun (*this, &EngineControl::control_app_button_clicked)); + control_app_button.set_name ("generic button"); + control_app_button.set_can_focus(true); manage_control_app_sensitivity (); - cancel_button = add_button (Gtk::Stock::CANCEL, Gtk::RESPONSE_CANCEL); - ok_button = add_button (Gtk::Stock::OK, Gtk::RESPONSE_OK); - apply_button = add_button (Gtk::Stock::APPLY, Gtk::RESPONSE_APPLY); + start_stop_button.signal_clicked.connect (mem_fun (*this, &EngineControl::start_stop_button_clicked)); + start_stop_button.set_sensitive (false); + start_stop_button.set_name ("generic button"); + start_stop_button.set_can_focus(true); + start_stop_button.set_can_default(true); + start_stop_button.set_act_on_release (false); + + update_devices_button.signal_clicked.connect (mem_fun (*this, &EngineControl::update_devices_button_clicked)); + update_devices_button.set_sensitive (false); + update_devices_button.set_name ("generic button"); + update_devices_button.set_can_focus(true); + + use_buffered_io_button.signal_clicked.connect (mem_fun (*this, &EngineControl::use_buffered_io_button_clicked)); + use_buffered_io_button.set_sensitive (false); + use_buffered_io_button.set_name ("generic button"); + use_buffered_io_button.set_can_focus(true); /* Pick up any existing audio setup configuration, if appropriate */ @@ -272,57 +301,179 @@ EngineControl::EngineControl () ARDOUR::AudioEngine::instance()->Running.connect (running_connection, MISSING_INVALIDATOR, boost::bind (&EngineControl::engine_running, this), gui_context()); ARDOUR::AudioEngine::instance()->Stopped.connect (stopped_connection, MISSING_INVALIDATOR, boost::bind (&EngineControl::engine_stopped, this), gui_context()); ARDOUR::AudioEngine::instance()->Halted.connect (stopped_connection, MISSING_INVALIDATOR, boost::bind (&EngineControl::engine_stopped, this), gui_context()); + ARDOUR::AudioEngine::instance()->DeviceListChanged.connect (devicelist_connection, MISSING_INVALIDATOR, boost::bind (&EngineControl::device_list_changed, this), gui_context()); - { - PBD::Unwinder protect_ignore_changes (ignore_changes, ignore_changes + 1); - backend_changed (); + if (audio_setup) { + if (!set_state (*audio_setup)) { + set_default_state (); + } + } else { + set_default_state (); } - maybe_display_saved_state(); - /* Connect to signals */ + update_sensitivity (); + connect_changed_signals (); + + notebook.signal_switch_page().connect (sigc::mem_fun (*this, &EngineControl::on_switch_page)); - driver_combo.signal_changed().connect (sigc::mem_fun (*this, &EngineControl::driver_changed)); - sample_rate_combo.signal_changed().connect (sigc::mem_fun (*this, &EngineControl::sample_rate_changed)); - buffer_size_combo.signal_changed().connect (sigc::mem_fun (*this, &EngineControl::buffer_size_changed)); - device_combo.signal_changed().connect (sigc::mem_fun (*this, &EngineControl::device_changed)); - midi_option_combo.signal_changed().connect (sigc::mem_fun (*this, &EngineControl::midi_option_changed)); + connect_disconnect_button.signal_clicked().connect (sigc::mem_fun (*this, &EngineControl::connect_disconnect_click)); - input_latency.signal_changed().connect (sigc::mem_fun (*this, &EngineControl::parameter_changed)); - output_latency.signal_changed().connect (sigc::mem_fun (*this, &EngineControl::parameter_changed)); - input_channels.signal_changed().connect (sigc::mem_fun (*this, &EngineControl::parameter_changed)); - output_channels.signal_changed().connect (sigc::mem_fun (*this, &EngineControl::parameter_changed)); + connect_disconnect_button.set_no_show_all(); + use_buffered_io_button.set_no_show_all(); + update_devices_button.set_no_show_all(); + start_stop_button.set_no_show_all(); + midi_devices_button.set_no_show_all(); +} - if (audio_setup) { - set_state (*audio_setup); +void +EngineControl::connect_changed_signals () +{ + backend_combo_connection = backend_combo.signal_changed ().connect ( + sigc::mem_fun (*this, &EngineControl::backend_changed)); + driver_combo_connection = driver_combo.signal_changed ().connect ( + sigc::mem_fun (*this, &EngineControl::driver_changed)); + sample_rate_combo_connection = sample_rate_combo.signal_changed ().connect ( + sigc::mem_fun (*this, &EngineControl::sample_rate_changed)); + buffer_size_combo_connection = buffer_size_combo.signal_changed ().connect ( + sigc::mem_fun (*this, &EngineControl::buffer_size_changed)); + nperiods_combo_connection = nperiods_combo.signal_changed ().connect ( + sigc::mem_fun (*this, &EngineControl::nperiods_changed)); + device_combo_connection = device_combo.signal_changed ().connect ( + sigc::mem_fun (*this, &EngineControl::device_changed)); + midi_option_combo_connection = midi_option_combo.signal_changed ().connect ( + sigc::mem_fun (*this, &EngineControl::midi_option_changed)); + + input_device_combo_connection = input_device_combo.signal_changed ().connect ( + sigc::mem_fun (*this, &EngineControl::input_device_changed)); + output_device_combo_connection = output_device_combo.signal_changed ().connect ( + sigc::mem_fun (*this, &EngineControl::output_device_changed)); + + input_latency_connection = input_latency.signal_changed ().connect ( + sigc::mem_fun (*this, &EngineControl::parameter_changed)); + output_latency_connection = output_latency.signal_changed ().connect ( + sigc::mem_fun (*this, &EngineControl::parameter_changed)); + input_channels_connection = input_channels.signal_changed ().connect ( + sigc::mem_fun (*this, &EngineControl::parameter_changed)); + output_channels_connection = output_channels.signal_changed ().connect ( + sigc::mem_fun (*this, &EngineControl::parameter_changed)); +} + +void +EngineControl::block_changed_signals () +{ + if (block_signals++ == 0) { + DEBUG_ECONTROL ("Blocking changed signals"); + backend_combo_connection.block (); + driver_combo_connection.block (); + sample_rate_combo_connection.block (); + buffer_size_combo_connection.block (); + nperiods_combo_connection.block (); + device_combo_connection.block (); + input_device_combo_connection.block (); + output_device_combo_connection.block (); + midi_option_combo_connection.block (); + input_latency_connection.block (); + output_latency_connection.block (); + input_channels_connection.block (); + output_channels_connection.block (); } +} - notebook.signal_switch_page().connect (sigc::mem_fun (*this, &EngineControl::on_switch_page)); +void +EngineControl::unblock_changed_signals () +{ + if (--block_signals == 0) { + DEBUG_ECONTROL ("Unblocking changed signals"); + backend_combo_connection.unblock (); + driver_combo_connection.unblock (); + sample_rate_combo_connection.unblock (); + buffer_size_combo_connection.unblock (); + nperiods_combo_connection.unblock (); + device_combo_connection.unblock (); + input_device_combo_connection.unblock (); + output_device_combo_connection.unblock (); + midi_option_combo_connection.unblock (); + input_latency_connection.unblock (); + output_latency_connection.unblock (); + input_channels_connection.unblock (); + output_channels_connection.unblock (); + } +} + +EngineControl::SignalBlocker::SignalBlocker (EngineControl& engine_control, + const std::string& reason) + : ec (engine_control) + , m_reason (reason) +{ + DEBUG_ECONTROL (string_compose ("SignalBlocker: %1", m_reason)); + ec.block_changed_signals (); +} + +EngineControl::SignalBlocker::~SignalBlocker () +{ + DEBUG_ECONTROL (string_compose ("~SignalBlocker: %1", m_reason)); + ec.unblock_changed_signals (); } void -EngineControl::on_response (int response_id) +EngineControl::on_show () { - ArdourDialog::on_response (response_id); + ArdourDialog::on_show (); + if (!ARDOUR::AudioEngine::instance()->current_backend() || !ARDOUR::AudioEngine::instance()->running()) { + // re-check _have_control (jackd running) see #6041 + backend_changed (); + } + device_changed (); + start_stop_button.grab_focus(); +} - switch (response_id) { - case RESPONSE_APPLY: - push_state_to_backend (true); - break; - case RESPONSE_OK: - push_state_to_backend (true); - hide (); - break; - case RESPONSE_DELETE_EVENT: - { - GdkEventButton ev; - ev.type = GDK_BUTTON_PRESS; - ev.button = 1; - on_delete_event ((GdkEventAny*) &ev); - break; - } - default: - hide (); +void +EngineControl::on_map () +{ + if (!ARDOUR_UI::instance()->session_loaded && !PublicEditor::_instance) { + set_type_hint (Gdk::WINDOW_TYPE_HINT_NORMAL); + } else if (UIConfiguration::instance().get_all_floating_windows_are_dialogs()) { + set_type_hint (Gdk::WINDOW_TYPE_HINT_DIALOG); + } else { + set_type_hint (Gdk::WINDOW_TYPE_HINT_UTILITY); } + ArdourDialog::on_map (); +} + +bool +EngineControl::try_autostart () +{ + if (!start_stop_button.get_sensitive()) { + return false; + } + if (ARDOUR::AudioEngine::instance()->running()) { + return true; + } + return start_engine (); +} + +bool +EngineControl::start_engine () +{ + if (push_state_to_backend(true) != 0) { + MessageDialog msg(*this, + ARDOUR::AudioEngine::instance()->get_last_backend_error()); + msg.run(); + return false; + } + return true; +} + +bool +EngineControl::stop_engine (bool for_latency) +{ + if (ARDOUR::AudioEngine::instance()->stop(for_latency)) { + MessageDialog msg(*this, + ARDOUR::AudioEngine::instance()->get_last_backend_error()); + msg.run(); + return false; + } + return true; } void @@ -344,6 +495,13 @@ EngineControl::build_notebook () basic_packer.attach (*label, 0, 1, 0, 1, xopt, (AttachOptions) 0); basic_packer.attach (backend_combo, 1, 2, 0, 1, xopt, (AttachOptions) 0); + basic_packer.attach (engine_status, 2, 3, 0, 1, xopt, (AttachOptions) 0); + engine_status.show(); + + basic_packer.attach (start_stop_button, 3, 4, 0, 1, xopt, xopt); + basic_packer.attach (update_devices_button, 3, 4, 1, 2, xopt, xopt); + basic_packer.attach (use_buffered_io_button, 3, 4, 2, 3, xopt, xopt); + lm_button_audio.signal_clicked.connect (sigc::mem_fun (*this, &EngineControl::calibrate_audio_latency)); lm_button_audio.set_name ("generic button"); lm_button_audio.set_can_focus(true); @@ -356,16 +514,10 @@ EngineControl::build_notebook () basic_vbox.pack_start (basic_hbox, false, false); - if (_have_control) { - Gtk::HBox* hpacker = manage (new HBox); - hpacker->set_border_width (12); - hpacker->pack_start (control_app_button, false, false); - hpacker->show (); - control_app_button.show(); - basic_vbox.pack_start (*hpacker); + { + PBD::Unwinder protect_ignore_changes (ignore_changes, ignore_changes + 1); + basic_vbox.show_all (); } - - basic_vbox.show_all (); } void @@ -389,10 +541,26 @@ EngineControl::build_full_control_notebook () row++; } - label = manage (left_aligned_label (_("Device:"))); - basic_packer.attach (*label, 0, 1, row, row + 1, xopt, (AttachOptions) 0); - basic_packer.attach (device_combo, 1, 2, row, row + 1, xopt, (AttachOptions) 0); - row++; + if (backend->use_separate_input_and_output_devices()) { + label = manage (left_aligned_label (_("Input Device:"))); + basic_packer.attach (*label, 0, 1, row, row + 1, xopt, (AttachOptions) 0); + basic_packer.attach (input_device_combo, 1, 2, row, row + 1, xopt, (AttachOptions) 0); + row++; + label = manage (left_aligned_label (_("Output Device:"))); + basic_packer.attach (*label, 0, 1, row, row + 1, xopt, (AttachOptions) 0); + basic_packer.attach (output_device_combo, 1, 2, row, row + 1, xopt, (AttachOptions) 0); + row++; + // reset so it isn't used in state comparisons + device_combo.set_active_text (""); + } else { + label = manage (left_aligned_label (_("Device:"))); + basic_packer.attach (*label, 0, 1, row, row + 1, xopt, (AttachOptions) 0); + basic_packer.attach (device_combo, 1, 2, row, row + 1, xopt, (AttachOptions) 0); + row++; + // reset these so they don't get used in state comparisons + input_device_combo.set_active_text (""); + output_device_combo.set_active_text (""); + } label = manage (left_aligned_label (_("Sample rate:"))); basic_packer.attach (*label, 0, 1, row, row + 1, xopt, (AttachOptions) 0); @@ -405,6 +573,19 @@ EngineControl::build_full_control_notebook () basic_packer.attach (buffer_size_combo, 1, 2, row, row + 1, xopt, (AttachOptions) 0); buffer_size_duration_label.set_alignment (0.0); /* left-align */ basic_packer.attach (buffer_size_duration_label, 2, 3, row, row+1, SHRINK, (AttachOptions) 0); + + int ctrl_btn_span = 1; + if (backend->can_set_period_size ()) { + row++; + label = manage (left_aligned_label (_("Periods:"))); + basic_packer.attach (*label, 0, 1, row, row + 1, xopt, (AttachOptions) 0); + basic_packer.attach (nperiods_combo, 1, 2, row, row + 1, xopt, (AttachOptions) 0); + ++ctrl_btn_span; + } + + /* button spans 2 or 3 rows */ + + basic_packer.attach (control_app_button, 3, 4, row - ctrl_btn_span, row + 1, xopt, xopt); row++; input_channels.set_name ("InputChannels"); @@ -413,10 +594,12 @@ EngineControl::build_full_control_notebook () input_channels.set_wrap (false); output_channels.set_editable (true); - label = manage (left_aligned_label (_("Input Channels:"))); - basic_packer.attach (*label, 0, 1, row, row+1, xopt, (AttachOptions) 0); - basic_packer.attach (input_channels, 1, 2, row, row+1, xopt, (AttachOptions) 0); - ++row; + if (!ARDOUR::Profile->get_mixbus()) { + label = manage (left_aligned_label (_("Input Channels:"))); + basic_packer.attach (*label, 0, 1, row, row+1, xopt, (AttachOptions) 0); + basic_packer.attach (input_channels, 1, 2, row, row+1, xopt, (AttachOptions) 0); + ++row; + } output_channels.set_name ("OutputChannels"); output_channels.set_flags (Gtk::CAN_FOCUS); @@ -424,10 +607,12 @@ EngineControl::build_full_control_notebook () output_channels.set_wrap (false); output_channels.set_editable (true); - label = manage (left_aligned_label (_("Output Channels:"))); - basic_packer.attach (*label, 0, 1, row, row+1, xopt, (AttachOptions) 0); - basic_packer.attach (output_channels, 1, 2, row, row+1, xopt, (AttachOptions) 0); - ++row; + if (!ARDOUR::Profile->get_mixbus()) { + label = manage (left_aligned_label (_("Output Channels:"))); + basic_packer.attach (*label, 0, 1, row, row+1, xopt, (AttachOptions) 0); + basic_packer.attach (output_channels, 1, 2, row, row+1, xopt, (AttachOptions) 0); + ++row; + } input_latency.set_name ("InputLatency"); input_latency.set_flags (Gtk::CAN_FOCUS); @@ -459,7 +644,7 @@ EngineControl::build_full_control_notebook () basic_packer.attach (lm_button_audio, 3, 4, row-1, row+1, xopt, xopt); ++row; - label = manage (left_aligned_label (_("MIDI System"))); + label = manage (left_aligned_label (_("MIDI System:"))); basic_packer.attach (*label, 0, 1, row, row + 1, xopt, (AttachOptions) 0); basic_packer.attach (midi_option_combo, 1, 2, row, row + 1, SHRINK, (AttachOptions) 0); basic_packer.attach (midi_devices_button, 3, 4, row, row+1, xopt, xopt); @@ -477,7 +662,7 @@ EngineControl::build_no_control_notebook () vector strings; AttachOptions xopt = AttachOptions (FILL|EXPAND); int row = 1; // row zero == backend combo - const string msg = string_compose (_("The %1 audio backend was configured and started externally.\nThis limits your control over it."), backend->name()); + const string msg = string_compose (_("%1 is already running. %2 will connect to it and use the existing settings."), backend->name(), PROGRAM_NAME); label = manage (new Label); label->set_markup (string_compose ("%1", msg)); @@ -500,8 +685,6 @@ EngineControl::build_no_control_notebook () row++; } - connect_disconnect_button.signal_clicked().connect (sigc::mem_fun (*this, &EngineControl::connect_disconnect_click)); - basic_packer.attach (connect_disconnect_button, 0, 2, row, row+1, FILL, AttachOptions (0)); row++; } @@ -531,35 +714,36 @@ EngineControl::enable_latency_tab () ARDOUR::AudioEngine::instance()->get_physical_outputs (type, outputs); ARDOUR::AudioEngine::instance()->get_physical_inputs (type, inputs); - if (_measure_midi) { - lm_preamble.set_markup (_("")); - } else { - lm_preamble.set_markup (_("Turn down the volume on your audio equipment to a very low level.")); + if (!ARDOUR::AudioEngine::instance()->running()) { + MessageDialog msg (_("Failed to start or connect to audio-engine.\n\nLatency calibration requires a working audio interface.")); + notebook.set_current_page (0); + msg.run (); + return; } - - if (inputs.empty() || outputs.empty()) { + else if (inputs.empty() || outputs.empty()) { MessageDialog msg (_("Your selected audio configuration is playback- or capture-only.\n\nLatency calibration requires playback and capture")); - lm_measure_button.set_sensitive (false); notebook.set_current_page (0); msg.run (); return; } - if (!outputs.empty()) { - set_popdown_strings (lm_output_channel_combo, outputs); - lm_output_channel_combo.set_active_text (outputs.front()); - lm_output_channel_combo.set_sensitive (true); + lm_back_button_signal.disconnect(); + if (_measure_midi) { + lm_back_button_signal = lm_back_button.signal_clicked().connect (sigc::bind (sigc::mem_fun (notebook, &Gtk::Notebook::set_current_page), midi_tab)); + lm_preamble.hide (); } else { - lm_output_channel_combo.set_sensitive (false); + lm_back_button_signal = lm_back_button.signal_clicked().connect( + sigc::mem_fun(*this, &EngineControl::latency_back_button_clicked)); + lm_preamble.show (); } - if (!inputs.empty()) { - set_popdown_strings (lm_input_channel_combo, inputs); - lm_input_channel_combo.set_active_text (inputs.front()); - lm_input_channel_combo.set_sensitive (true); - } else { - lm_input_channel_combo.set_sensitive (false); - } + set_popdown_strings (lm_output_channel_combo, outputs); + lm_output_channel_combo.set_active_text (outputs.front()); + lm_output_channel_combo.set_sensitive (true); + + set_popdown_strings (lm_input_channel_combo, inputs); + lm_input_channel_combo.set_active_text (inputs.front()); + lm_input_channel_combo.set_sensitive (true); lm_measure_button.set_sensitive (true); } @@ -583,6 +767,129 @@ EngineControl::setup_midi_tab_for_backend () midi_vbox.show_all (); } +void +EngineControl::update_sensitivity () +{ + boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); + if (!backend) { + start_stop_button.set_sensitive (false); + return; + } + + bool valid = true; + size_t devices_available = 0; + bool engine_running = ARDOUR::AudioEngine::instance()->running(); + + if (backend->use_separate_input_and_output_devices ()) { + devices_available += get_popdown_string_count (input_device_combo); + devices_available += get_popdown_string_count (output_device_combo); + } else { + devices_available += get_popdown_string_count (device_combo); + } + + if (devices_available == 0) { + valid = false; + input_latency.set_sensitive (false); + output_latency.set_sensitive (false); + input_channels.set_sensitive (false); + output_channels.set_sensitive (false); + } else { + input_latency.set_sensitive (true); + output_latency.set_sensitive (true); + input_channels.set_sensitive (!engine_running); + output_channels.set_sensitive (!engine_running); + } + + if (get_popdown_string_count (buffer_size_combo) > 0) { + if (!engine_running) { + buffer_size_combo.set_sensitive (valid); + } else if (backend->can_change_buffer_size_when_running ()) { + buffer_size_combo.set_sensitive (valid || !_have_control); + } else { + buffer_size_combo.set_sensitive (false); + } + } else { + buffer_size_combo.set_sensitive (false); + valid = false; + } + + if (get_popdown_string_count (sample_rate_combo) > 0) { + bool allow_to_set_rate = false; + if (!engine_running) { + if (!ARDOUR_UI::instance()->session_loaded) { + // engine is not running, no session loaded -> anything goes. + allow_to_set_rate = true; + } else if (_desired_sample_rate > 0 && get_rate () != _desired_sample_rate) { + // only allow to change if the current setting is not the native session rate. + allow_to_set_rate = true; + } + } + sample_rate_combo.set_sensitive (allow_to_set_rate); + } else { + sample_rate_combo.set_sensitive (false); + valid = false; + } + + if (get_popdown_string_count (nperiods_combo) > 0) { + if (!engine_running) { + nperiods_combo.set_sensitive (true); + } else { + nperiods_combo.set_sensitive (false); + } + } else { + nperiods_combo.set_sensitive (false); + } + + if (_have_control) { + start_stop_button.set_sensitive(true); + start_stop_button.show(); + if (engine_running) { + start_stop_button.set_text("Stop"); + update_devices_button.set_sensitive(false); + use_buffered_io_button.set_sensitive(false); + } else { + if (backend->can_request_update_devices()) { + update_devices_button.show(); + } else { + update_devices_button.hide(); + } + if (backend->can_use_buffered_io()) { + use_buffered_io_button.show(); + } else { + use_buffered_io_button.hide(); + } + start_stop_button.set_text("Start"); + update_devices_button.set_sensitive(true); + use_buffered_io_button.set_sensitive(true); + } + } else { + update_devices_button.set_sensitive(false); + update_devices_button.hide(); + use_buffered_io_button.set_sensitive(false); + use_buffered_io_button.hide(); + start_stop_button.set_sensitive(false); + start_stop_button.hide(); + } + + if (engine_running && _have_control) { + input_device_combo.set_sensitive (false); + output_device_combo.set_sensitive (false); + device_combo.set_sensitive (false); + driver_combo.set_sensitive (false); + } else { + input_device_combo.set_sensitive (true); + output_device_combo.set_sensitive (true); + device_combo.set_sensitive (true); + if (backend->requires_driver_selection() && get_popdown_string_count(driver_combo) > 0) { + driver_combo.set_sensitive (true); + } else { + driver_combo.set_sensitive (false); + } + } + + midi_option_combo.set_sensitive (!engine_running); +} + void EngineControl::setup_midi_tab_for_jack () { @@ -677,22 +984,21 @@ EngineControl::refresh_midi_display (std::string focus) } } -void -EngineControl::update_sensitivity () -{ -} - void EngineControl::backend_changed () { + SignalBlocker blocker (*this, "backend_changed"); string backend_name = backend_combo.get_active_text(); boost::shared_ptr backend; - if (!(backend = ARDOUR::AudioEngine::instance()->set_backend (backend_name, "ardour", ""))) { + if (!(backend = ARDOUR::AudioEngine::instance()->set_backend (backend_name, downcase (std::string(PROGRAM_NAME)), ""))) { /* eh? setting the backend failed... how ? */ + /* A: stale config contains a backend that does not exist in current build */ return; } + DEBUG_ECONTROL (string_compose ("Backend name: %1", backend_name)); + _have_control = ARDOUR::AudioEngine::instance()->setup_required (); build_notebook (); @@ -700,27 +1006,59 @@ EngineControl::backend_changed () _midi_devices.clear(); if (backend->requires_driver_selection()) { - vector drivers = backend->enumerate_drivers(); - driver_combo.set_sensitive (true); - - if (!drivers.empty()) { - { - PBD::Unwinder protect_ignore_changes (ignore_changes, ignore_changes + 1); - set_popdown_strings (driver_combo, drivers); - driver_combo.set_active_text (drivers.front()); - } - + if (set_driver_popdown_strings ()) { driver_changed (); } - } else { - driver_combo.set_sensitive (false); /* this will change the device text which will cause a call to * device changed which will set up parameters */ list_devices (); } + update_midi_options (); + + connect_disconnect_button.hide(); + + midi_option_changed(); + + started_at_least_once = false; + + /* changing the backend implies stopping the engine + * ARDOUR::AudioEngine() may or may not emit this signal + * depending on previous engine state + */ + engine_stopped (); // set "active/inactive" + + if (!_have_control) { + // set settings from backend that we do have control over + set_buffersize_popdown_strings (); + set_active_text_if_present (buffer_size_combo, bufsize_as_string (backend->buffer_size())); + } + + if (_have_control && !ignore_changes) { + // set driver & devices + State state = get_matching_state (backend_combo.get_active_text()); + if (state) { + DEBUG_ECONTROL ("backend-changed(): found prior state for backend"); + PBD::Unwinder protect_ignore_changes (ignore_changes, ignore_changes + 1); + set_current_state (state); + } else { + DEBUG_ECONTROL ("backend-changed(): no prior state for backend"); + } + } else { + DEBUG_ECONTROL (string_compose ("backend-changed(): _have_control=%1 ignore_changes=%2", _have_control, ignore_changes)); + } + + if (!ignore_changes) { + maybe_display_saved_state (); + } +} + +void +EngineControl::update_midi_options () +{ + boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); vector midi_options = backend->enumerate_midi_options(); if (midi_options.size() == 1) { @@ -735,19 +1073,15 @@ EngineControl::backend_changed () midi_option_combo.set_sensitive (false); } } - - midi_option_changed(); - - started_at_least_once = false; - - if (!ignore_changes) { - maybe_display_saved_state (); - } } bool EngineControl::print_channel_count (Gtk::SpinButton* sb) { + if (ARDOUR::Profile->get_mixbus()) { + return true; + } + uint32_t cnt = (uint32_t) sb->get_value(); if (cnt == 0) { sb->set_text (_("all available channels")); @@ -759,14 +1093,82 @@ EngineControl::print_channel_count (Gtk::SpinButton* sb) return true; } -void -EngineControl::list_devices () +// @return true if there are drivers available +bool +EngineControl::set_driver_popdown_strings () { + DEBUG_ECONTROL ("set_driver_popdown_strings"); boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); - assert (backend); + vector drivers = backend->enumerate_drivers(); - /* now fill out devices, mark sample rates, buffer sizes insensitive */ + if (drivers.empty ()) { + // This is an error...? + return false; + } + string current_driver = backend->driver_name (); + + DEBUG_ECONTROL (string_compose ("backend->driver_name: %1", current_driver)); + + if (std::find (drivers.begin (), drivers.end (), current_driver) == + drivers.end ()) { + + current_driver = drivers.front (); + } + + set_popdown_strings (driver_combo, drivers); + DEBUG_ECONTROL ( + string_compose ("driver_combo.set_active_text: %1", current_driver)); + driver_combo.set_active_text (current_driver); + return true; +} + +std::string +EngineControl::get_default_device(const string& current_device_name, + const vector& available_devices) +{ + // If the current device is available, use it as default + if (std::find (available_devices.begin (), + available_devices.end (), + current_device_name) != available_devices.end ()) { + + return current_device_name; + } + + using namespace ARDOUR; + + string default_device_name = + AudioBackend::get_standard_device_name(AudioBackend::DeviceDefault); + + vector::const_iterator i; + + // If there is a "Default" device available, use it + for (i = available_devices.begin(); i != available_devices.end(); ++i) { + if (*i == default_device_name) { + return *i; + } + } + + string none_device_name = + AudioBackend::get_standard_device_name(AudioBackend::DeviceNone); + + // Use the first device that isn't "None" + for (i = available_devices.begin(); i != available_devices.end(); ++i) { + if (*i != none_device_name) { + return *i; + } + } + + // Use "None" if there are no other available + return available_devices.front(); +} + +// @return true if there are devices available +bool +EngineControl::set_device_popdown_strings () +{ + DEBUG_ECONTROL ("set_device_popdown_strings"); + boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); vector all_devices = backend->enumerate_devices (); /* NOTE: Ardour currently does not display the "available" field of the @@ -784,147 +1186,367 @@ EngineControl::list_devices () available_devices.push_back (i->name); } - if (!available_devices.empty()) { + if (available_devices.empty ()) { + return false; + } - update_sensitivity (); + set_popdown_strings (device_combo, available_devices); - { - PBD::Unwinder protect_ignore_changes (ignore_changes, ignore_changes + 1); - set_popdown_strings (device_combo, available_devices); - device_combo.set_active_text (available_devices.front()); - } + std::string default_device = + get_default_device(backend->device_name(), available_devices); - device_changed (); + DEBUG_ECONTROL ( + string_compose ("set device_combo active text: %1", default_device)); + + device_combo.set_active_text(default_device); + return true; +} + +// @return true if there are input devices available +bool +EngineControl::set_input_device_popdown_strings () +{ + DEBUG_ECONTROL ("set_input_device_popdown_strings"); + boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); + vector all_devices = backend->enumerate_input_devices (); + + vector available_devices; + + for (vector::const_iterator i = all_devices.begin(); i != all_devices.end(); ++i) { + available_devices.push_back (i->name); + } + + if (available_devices.empty()) { + return false; + } + + set_popdown_strings (input_device_combo, available_devices); + + std::string default_device = + get_default_device(backend->input_device_name(), available_devices); + + DEBUG_ECONTROL ( + string_compose ("set input_device_combo active text: %1", default_device)); + input_device_combo.set_active_text(default_device); + return true; +} + +// @return true if there are output devices available +bool +EngineControl::set_output_device_popdown_strings () +{ + DEBUG_ECONTROL ("set_output_device_popdown_strings"); + boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); + vector all_devices = backend->enumerate_output_devices (); + + vector available_devices; - ok_button->set_sensitive (true); - apply_button->set_sensitive (true); + for (vector::const_iterator i = all_devices.begin(); i != all_devices.end(); ++i) { + available_devices.push_back (i->name); + } + + if (available_devices.empty()) { + return false; + } + + set_popdown_strings (output_device_combo, available_devices); + + std::string default_device = + get_default_device(backend->output_device_name(), available_devices); + DEBUG_ECONTROL ( + string_compose ("set output_device_combo active text: %1", default_device)); + output_device_combo.set_active_text(default_device); + return true; +} + +void +EngineControl::list_devices () +{ + DEBUG_ECONTROL ("list_devices"); + boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); + assert (backend); + + /* now fill out devices, mark sample rates, buffer sizes insensitive */ + + bool devices_available = false; + + if (backend->use_separate_input_and_output_devices ()) { + bool input_devices_available = set_input_device_popdown_strings (); + bool output_devices_available = set_output_device_popdown_strings (); + devices_available = input_devices_available || output_devices_available; + } else { + devices_available = set_device_popdown_strings (); + } + + if (devices_available) { + device_changed (); } else { device_combo.clear(); - sample_rate_combo.set_sensitive (false); - buffer_size_combo.set_sensitive (false); - input_latency.set_sensitive (false); - output_latency.set_sensitive (false); - input_channels.set_sensitive (false); - output_channels.set_sensitive (false); - if (_have_control) { - ok_button->set_sensitive (false); - apply_button->set_sensitive (false); - } else { - ok_button->set_sensitive (true); - apply_button->set_sensitive (true); - } + input_device_combo.clear(); + output_device_combo.clear(); } + update_sensitivity (); } void EngineControl::driver_changed () { + SignalBlocker blocker (*this, "driver_changed"); boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); assert (backend); backend->set_driver (driver_combo.get_active_text()); list_devices (); + // TODO load LRU device(s) for backend + driver combo + if (!ignore_changes) { maybe_display_saved_state (); } } -void -EngineControl::device_changed () +vector +EngineControl::get_sample_rates_for_all_devices () +{ + boost::shared_ptr backend = + ARDOUR::AudioEngine::instance ()->current_backend (); + vector all_rates; + + if (backend->use_separate_input_and_output_devices ()) { + all_rates = backend->available_sample_rates2 (get_input_device_name (), get_output_device_name ()); + } else { + all_rates = backend->available_sample_rates (get_device_name ()); + } + return all_rates; +} + +vector +EngineControl::get_default_sample_rates () { + vector rates; + rates.push_back (8000.0f); + rates.push_back (16000.0f); + rates.push_back (32000.0f); + rates.push_back (44100.0f); + rates.push_back (48000.0f); + rates.push_back (88200.0f); + rates.push_back (96000.0f); + rates.push_back (192000.0f); + rates.push_back (384000.0f); + return rates; +} +void +EngineControl::set_samplerate_popdown_strings () +{ + DEBUG_ECONTROL ("set_samplerate_popdown_strings"); boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); - assert (backend); - string device_name = device_combo.get_active_text (); + string desired; + vector sr; vector s; - { - PBD::Unwinder protect_ignore_changes (ignore_changes, ignore_changes + 1); + if (_have_control) { + sr = get_sample_rates_for_all_devices (); + } else { + sr = get_default_sample_rates (); + } - /* don't allow programmatic change to combos to cause a - recursive call to this method. - */ + for (vector::const_iterator x = sr.begin(); x != sr.end(); ++x) { + s.push_back (rate_as_string (*x)); + if (*x == _desired_sample_rate) { + desired = s.back(); + } + } - /* sample rates */ + set_popdown_strings (sample_rate_combo, s); - string desired; + if (!s.empty()) { + if (ARDOUR::AudioEngine::instance()->running()) { + sample_rate_combo.set_active_text (rate_as_string (backend->sample_rate())); + } else if (ARDOUR_UI::instance()->session_loaded) { + float active_sr = ARDOUR_UI::instance()->the_session()->nominal_sample_rate (); - vector sr; + if (std::find (sr.begin (), sr.end (), active_sr) == sr.end ()) { + active_sr = sr.front (); + } - if (_have_control) { - sr = backend->available_sample_rates (device_name); - } else { + sample_rate_combo.set_active_text (rate_as_string (active_sr)); + } else if (desired.empty ()) { + float new_active_sr = backend->default_sample_rate (); - sr.push_back (8000.0f); - sr.push_back (16000.0f); - sr.push_back (32000.0f); - sr.push_back (44100.0f); - sr.push_back (48000.0f); - sr.push_back (88200.0f); - sr.push_back (96000.0f); - sr.push_back (192000.0f); - sr.push_back (384000.0f); - } - - for (vector::const_iterator x = sr.begin(); x != sr.end(); ++x) { - s.push_back (rate_as_string (*x)); - if (*x == _desired_sample_rate) { - desired = s.back(); + if (std::find (sr.begin (), sr.end (), new_active_sr) == sr.end ()) { + new_active_sr = sr.front (); } + + sample_rate_combo.set_active_text (rate_as_string (new_active_sr)); + } else { + sample_rate_combo.set_active_text (desired); } + } + update_sensitivity (); +} - if (!s.empty()) { - sample_rate_combo.set_sensitive (true); - set_popdown_strings (sample_rate_combo, s); +vector +EngineControl::get_buffer_sizes_for_all_devices () +{ + boost::shared_ptr backend = + ARDOUR::AudioEngine::instance ()->current_backend (); + vector all_sizes; - if (desired.empty()) { - sample_rate_combo.set_active_text (rate_as_string (backend->default_sample_rate())); - } else { - sample_rate_combo.set_active_text (desired); - } + if (backend->use_separate_input_and_output_devices ()) { + all_sizes = backend->available_buffer_sizes2 (get_input_device_name (), get_output_device_name ()); + } else { + all_sizes = backend->available_buffer_sizes (get_device_name ()); + } + return all_sizes; +} + +vector +EngineControl::get_default_buffer_sizes () +{ + vector sizes; + sizes.push_back (8); + sizes.push_back (16); + sizes.push_back (32); + sizes.push_back (64); + sizes.push_back (128); + sizes.push_back (256); + sizes.push_back (512); + sizes.push_back (1024); + sizes.push_back (2048); + sizes.push_back (4096); + sizes.push_back (8192); + return sizes; +} + +void +EngineControl::set_buffersize_popdown_strings () +{ + DEBUG_ECONTROL ("set_buffersize_popdown_strings"); + boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); + vector bs; + vector s; + + if (_have_control) { + bs = get_buffer_sizes_for_all_devices (); + } else if (backend->can_change_buffer_size_when_running()) { + bs = get_default_buffer_sizes (); + } + + for (vector::const_iterator x = bs.begin(); x != bs.end(); ++x) { + s.push_back (bufsize_as_string (*x)); + } + + uint32_t previous_size = backend->buffer_size (); + if (!buffer_size_combo.get_active_text().empty()) { + previous_size = get_buffer_size (); + } + set_popdown_strings (buffer_size_combo, s); + + if (!s.empty()) { + + if (std::find(bs.begin(), bs.end(), previous_size) != bs.end()) { + buffer_size_combo.set_active_text(bufsize_as_string(previous_size)); } else { - sample_rate_combo.set_sensitive (false); + + buffer_size_combo.set_active_text(s.front()); + + uint32_t period = backend->buffer_size(); + if (0 == period && backend->use_separate_input_and_output_devices()) { + period = backend->default_buffer_size(get_input_device_name()); + } + if (0 == period && backend->use_separate_input_and_output_devices()) { + period = backend->default_buffer_size(get_output_device_name()); + } + if (0 == period && !backend->use_separate_input_and_output_devices()) { + period = backend->default_buffer_size(get_device_name()); + } + + set_active_text_if_present(buffer_size_combo, bufsize_as_string(period)); } + show_buffer_duration (); + } + update_sensitivity (); +} + +void +EngineControl::set_nperiods_popdown_strings () +{ + DEBUG_ECONTROL ("set_nperiods_popdown_strings"); + boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); + vector np; + vector s; - /* buffer sizes */ + if (backend->can_set_period_size()) { + np = backend->available_period_sizes (get_driver()); + } - vector bs; + for (vector::const_iterator x = np.begin(); x != np.end(); ++x) { + s.push_back (to_string (*x)); + } - if (_have_control) { - bs = backend->available_buffer_sizes (device_name); - } else if (backend->can_change_buffer_size_when_running()) { - bs.push_back (8); - bs.push_back (16); - bs.push_back (32); - bs.push_back (64); - bs.push_back (128); - bs.push_back (256); - bs.push_back (512); - bs.push_back (1024); - bs.push_back (2048); - bs.push_back (4096); - bs.push_back (8192); - } - s.clear (); - for (vector::const_iterator x = bs.begin(); x != bs.end(); ++x) { - s.push_back (bufsize_as_string (*x)); - } - - if (!s.empty()) { - buffer_size_combo.set_sensitive (true); - set_popdown_strings (buffer_size_combo, s); - - buffer_size_combo.set_active_text (bufsize_as_string (backend->default_buffer_size())); - show_buffer_duration (); - } else { - buffer_size_combo.set_sensitive (false); + set_popdown_strings (nperiods_combo, s); + + if (!s.empty()) { + set_active_text_if_present (nperiods_combo, to_string (backend->period_size())); // XXX + } + + update_sensitivity (); +} + +void +EngineControl::device_changed () +{ + SignalBlocker blocker (*this, "device_changed"); + boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); + assert (backend); + + string device_name_in; + string device_name_out; // only used if backend support separate I/O devices + + if (backend->use_separate_input_and_output_devices()) { + device_name_in = get_input_device_name (); + device_name_out = get_output_device_name (); + } else { + device_name_in = get_device_name (); + } + + /* we set the backend-device to query various device related intormation. + * This has the side effect that backend->device_name() will match + * the device_name and 'change_device' will never be true. + * so work around this by setting... + */ + if (backend->use_separate_input_and_output_devices()) { + if (device_name_in != backend->input_device_name() || device_name_out != backend->output_device_name ()) { + queue_device_changed = true; + } + } else { + if (device_name_in != backend->device_name()) { + queue_device_changed = true; } + } + + //the device name must be set FIRST so ASIO can populate buffersizes and the control panel button + if (backend->use_separate_input_and_output_devices()) { + backend->set_input_device_name (device_name_in); + backend->set_output_device_name (device_name_out); + } else { + backend->set_device_name(device_name_in); + } + + { + /* don't allow programmatic change to combos to cause a + recursive call to this method. + */ + PBD::Unwinder protect_ignore_changes (ignore_changes, ignore_changes + 1); + + set_samplerate_popdown_strings (); + set_buffersize_popdown_strings (); + set_nperiods_popdown_strings (); - /* XXX theoretically need to set min + max channel counts here - */ + /* TODO set min + max channel counts here */ manage_control_app_sensitivity (); } @@ -936,44 +1558,97 @@ EngineControl::device_changed () } } +void +EngineControl::input_device_changed () +{ + DEBUG_ECONTROL ("input_device_changed"); + + boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); + if (backend && backend->match_input_output_devices_or_none ()) { + const std::string& dev_none = ARDOUR::AudioBackend::get_standard_device_name (ARDOUR::AudioBackend::DeviceNone); + + if (get_output_device_name () != dev_none + && get_input_device_name () != dev_none + && get_input_device_name () != get_output_device_name ()) { + block_changed_signals (); + if (contains_value (output_device_combo, get_input_device_name ())) { + output_device_combo.set_active_text (get_input_device_name ()); + } else { + assert (contains_value (output_device_combo, dev_none)); + output_device_combo.set_active_text (dev_none); + } + unblock_changed_signals (); + } + } + device_changed (); +} + +void +EngineControl::output_device_changed () +{ + DEBUG_ECONTROL ("output_device_changed"); + boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); + if (backend && backend->match_input_output_devices_or_none ()) { + const std::string& dev_none = ARDOUR::AudioBackend::get_standard_device_name (ARDOUR::AudioBackend::DeviceNone); + + if (get_input_device_name () != dev_none + && get_input_device_name () != dev_none + && get_input_device_name () != get_output_device_name ()) { + block_changed_signals (); + if (contains_value (input_device_combo, get_output_device_name ())) { + input_device_combo.set_active_text (get_output_device_name ()); + } else { + assert (contains_value (input_device_combo, dev_none)); + input_device_combo.set_active_text (dev_none); + } + unblock_changed_signals (); + } + } + device_changed (); +} + string EngineControl::bufsize_as_string (uint32_t sz) { - /* Translators: "samples" is always plural here, so no - need for plural+singular forms. - */ - char buf[32]; - snprintf (buf, sizeof (buf), _("%u samples"), sz); - return buf; + return string_compose (P_("%1 sample", "%1 samples", sz), to_string(sz)); } void EngineControl::sample_rate_changed () { + DEBUG_ECONTROL ("sample_rate_changed"); /* reset the strings for buffer size to show the correct msec value (reflecting the new sample rate). */ show_buffer_duration (); - if (!ignore_changes) { - save_state (); - } } void EngineControl::buffer_size_changed () { - show_buffer_duration (); - if (!ignore_changes) { - save_state (); + DEBUG_ECONTROL ("buffer_size_changed"); + if (ARDOUR::AudioEngine::instance()->running()) { + boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); + if (backend && backend->can_change_buffer_size_when_running ()) { + backend->set_buffer_size (get_buffer_size()); + } } + show_buffer_duration (); } void -EngineControl::show_buffer_duration () +EngineControl::nperiods_changed () { + DEBUG_ECONTROL ("nperiods_changed"); + show_buffer_duration (); +} +void +EngineControl::show_buffer_duration () +{ + DEBUG_ECONTROL ("show_buffer_duration"); /* buffer sizes - convert from just samples to samples + msecs for * the displayed string */ @@ -982,21 +1657,27 @@ EngineControl::show_buffer_duration () uint32_t samples = atoi (bs_text); /* will ignore trailing text */ uint32_t rate = get_rate(); - /* Translators: "msecs" is ALWAYS plural here, so we do not - need singular form as well. - */ - /* Developers: note the hard-coding of a double buffered model - in the (2 * samples) computation of latency. we always start - the audiobackend in this configuration. + /* Except for ALSA and Dummy backends, we don't know the number of periods + * per cycle and settings. + * + * jack1 vs jack2 have different default latencies since jack2 start + * in async-mode unless --sync is given which adds an extra cycle + * of latency. The value is not known if jackd is started externally.. + * + * So just display the period size, that's also what + * ARDOUR_UI::update_sample_rate() does for the status bar. + * (the statusbar calls AudioEngine::instance()->usecs_per_cycle() + * but still, that's the buffer period, not [round-trip] latency) */ char buf[32]; - snprintf (buf, sizeof (buf), _("(%.1f msecs)"), (2 * samples) / (rate/1000.0)); + snprintf (buf, sizeof (buf), _("(%.1f ms)"), (samples / (rate/1000.0f))); buffer_size_duration_label.set_text (buf); } void EngineControl::midi_option_changed () { + DEBUG_ECONTROL ("midi_option_changed"); boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); assert (backend); @@ -1027,22 +1708,26 @@ EngineControl::midi_option_changed () _midi_devices = new_devices; if (_midi_devices.empty()) { - midi_devices_button.set_sensitive (false); + midi_devices_button.hide (); } else { - midi_devices_button.set_sensitive (true); - } - - if (!ignore_changes) { - save_state (); + midi_devices_button.show (); } } void EngineControl::parameter_changed () { - if (!ignore_changes) { - save_state (); +} + +EngineControl::State +EngineControl::get_matching_state (const string& backend) +{ + for (StateList::iterator i = states.begin(); i != states.end(); ++i) { + if ((*i)->backend == backend) { + return (*i); + } } + return State(); } EngineControl::State @@ -1053,8 +1738,25 @@ EngineControl::get_matching_state ( { for (StateList::iterator i = states.begin(); i != states.end(); ++i) { if ((*i)->backend == backend && - (*i)->driver == driver && - (*i)->device == device) { + (!_have_control || ((*i)->driver == driver && (*i)->device == device))) + { + return (*i); + } + } + return State(); +} + +EngineControl::State +EngineControl::get_matching_state ( + const string& backend, + const string& driver, + const string& input_device, + const string& output_device) +{ + for (StateList::iterator i = states.begin(); i != states.end(); ++i) { + if ((*i)->backend == backend && + (!_have_control || ((*i)->driver == driver && ((*i)->input_device == input_device) && (*i)->output_device == output_device))) + { return (*i); } } @@ -1067,30 +1769,70 @@ EngineControl::get_saved_state_for_currently_displayed_backend_and_device () boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); if (backend) { - return get_matching_state (backend_combo.get_active_text(), - (backend->requires_driver_selection() ? (std::string) driver_combo.get_active_text() : string()), - device_combo.get_active_text()); + if (backend->use_separate_input_and_output_devices ()) { + return get_matching_state (backend_combo.get_active_text(), + (backend->requires_driver_selection() ? (std::string) driver_combo.get_active_text() : string()), + input_device_combo.get_active_text(), + output_device_combo.get_active_text()); + } else { + return get_matching_state (backend_combo.get_active_text(), + (backend->requires_driver_selection() ? (std::string) driver_combo.get_active_text() : string()), + device_combo.get_active_text()); + } } - return get_matching_state (backend_combo.get_active_text(), string(), device_combo.get_active_text()); } +bool EngineControl::equivalent_states (const EngineControl::State& state1, + const EngineControl::State& state2) +{ + if (state1->backend == state2->backend && + state1->driver == state2->driver && + state1->device == state2->device && + state1->input_device == state2->input_device && + state1->output_device == state2->output_device) { + return true; + } + return false; +} + +// sort active first, then most recently used to the beginning of the list +bool +EngineControl::state_sort_cmp (const State &a, const State &b) { + if (a->active) { + return true; + } + else if (b->active) { + return false; + } + else { + return a->lru > b->lru; + } +} + EngineControl::State EngineControl::save_state () { + State state; + if (!_have_control) { - return State(); + state = get_matching_state (backend_combo.get_active_text(), string(), string()); + if (state) { + state->lru = time (NULL) ; + return state; + } + state.reset(new StateStruct); + state->backend = get_backend (); + } else { + state.reset(new StateStruct); + store_state (state); } - State state (new StateStruct); - store_state (state); for (StateList::iterator i = states.begin(); i != states.end();) { - if ((*i)->backend == state->backend && - (*i)->driver == state->driver && - (*i)->device == state->device) { + if (equivalent_states (*i, state)) { i = states.erase(i); } else { ++i; @@ -1099,6 +1841,8 @@ EngineControl::save_state () states.push_back (state); + states.sort (state_sort_cmp); + return state; } @@ -1108,32 +1852,40 @@ EngineControl::store_state (State state) state->backend = get_backend (); state->driver = get_driver (); state->device = get_device_name (); + state->input_device = get_input_device_name (); + state->output_device = get_output_device_name (); state->sample_rate = get_rate (); state->buffer_size = get_buffer_size (); + state->n_periods = get_nperiods (); state->input_latency = get_input_latency (); state->output_latency = get_output_latency (); state->input_channels = get_input_channels (); state->output_channels = get_output_channels (); state->midi_option = get_midi_option (); state->midi_devices = _midi_devices; + state->use_buffered_io = get_use_buffered_io (); + state->lru = time (NULL) ; } void EngineControl::maybe_display_saved_state () { - if (!_have_control) { + if (!_have_control || ARDOUR::AudioEngine::instance()->running ()) { return; } State state = get_saved_state_for_currently_displayed_backend_and_device (); if (state) { + DEBUG_ECONTROL ("Restoring saved state"); PBD::Unwinder protect_ignore_changes (ignore_changes, ignore_changes + 1); - if (!_desired_sample_rate) { + if (0 == _desired_sample_rate && sample_rate_combo.get_sensitive ()) { sample_rate_combo.set_active_text (rate_as_string (state->sample_rate)); } - buffer_size_combo.set_active_text (bufsize_as_string (state->buffer_size)); + set_active_text_if_present (buffer_size_combo, bufsize_as_string (state->buffer_size)); + + set_active_text_if_present (nperiods_combo, to_string(state->n_periods)); /* call this explicitly because we're ignoring changes to the controls at this point. */ @@ -1141,16 +1893,22 @@ EngineControl::maybe_display_saved_state () input_latency.set_value (state->input_latency); output_latency.set_value (state->output_latency); + use_buffered_io_button.set_active (state->use_buffered_io); + if (!state->midi_option.empty()) { midi_option_combo.set_active_text (state->midi_option); _midi_devices = state->midi_devices; } + } else { + DEBUG_ECONTROL ("Unable to find saved state for backend and devices"); } } XMLNode& EngineControl::get_state () { + LocaleGuard lg; + XMLNode* root = new XMLNode ("AudioMIDISetup"); std::string path; @@ -1161,25 +1919,31 @@ EngineControl::get_state () XMLNode* node = new XMLNode ("State"); - node->add_property ("backend", (*i)->backend); - node->add_property ("driver", (*i)->driver); - node->add_property ("device", (*i)->device); - node->add_property ("sample-rate", (*i)->sample_rate); - node->add_property ("buffer-size", (*i)->buffer_size); - node->add_property ("input-latency", (*i)->input_latency); - node->add_property ("output-latency", (*i)->output_latency); - node->add_property ("input-channels", (*i)->input_channels); - node->add_property ("output-channels", (*i)->output_channels); - node->add_property ("active", (*i)->active ? "yes" : "no"); - node->add_property ("midi-option", (*i)->midi_option); + node->set_property ("backend", (*i)->backend); + node->set_property ("driver", (*i)->driver); + node->set_property ("device", (*i)->device); + node->set_property ("input-device", (*i)->input_device); + node->set_property ("output-device", (*i)->output_device); + node->set_property ("sample-rate", (*i)->sample_rate); + node->set_property ("buffer-size", (*i)->buffer_size); + node->set_property ("n-periods", (*i)->n_periods); + node->set_property ("input-latency", (*i)->input_latency); + node->set_property ("output-latency", (*i)->output_latency); + node->set_property ("input-channels", (*i)->input_channels); + node->set_property ("output-channels", (*i)->output_channels); + node->set_property ("active", (*i)->active); + node->set_property ("use-buffered-io", (*i)->use_buffered_io); + node->set_property ("midi-option", (*i)->midi_option); + int32_t lru_val = (*i)->active ? time (NULL) : (*i)->lru; + node->set_property ("lru", lru_val ); XMLNode* midi_devices = new XMLNode ("MIDIDevices"); for (std::vector::const_iterator p = (*i)->midi_devices.begin(); p != (*i)->midi_devices.end(); ++p) { XMLNode* midi_device_stuff = new XMLNode ("MIDIDevice"); - midi_device_stuff->add_property (X_("name"), (*p)->name); - midi_device_stuff->add_property (X_("enabled"), (*p)->enabled); - midi_device_stuff->add_property (X_("input-latency"), (*p)->input_latency); - midi_device_stuff->add_property (X_("output-latency"), (*p)->output_latency); + midi_device_stuff->set_property (X_("name"), (*p)->name); + midi_device_stuff->set_property (X_("enabled"), (*p)->enabled); + midi_device_stuff->set_property (X_("input-latency"), (*p)->input_latency); + midi_device_stuff->set_property (X_("output-latency"), (*p)->output_latency); midi_devices->add_child_nocopy (*midi_device_stuff); } node->add_child_nocopy (*midi_devices); @@ -1194,16 +1958,31 @@ EngineControl::get_state () } void +EngineControl::set_default_state () +{ + vector backend_names; + vector backends = ARDOUR::AudioEngine::instance()->available_backends(); + + for (vector::const_iterator b = backends.begin(); b != backends.end(); ++b) { + backend_names.push_back ((*b)->name); + } + backend_combo.set_active_text (backend_names.front()); + + // We could set default backends per platform etc here + + backend_changed (); +} + +bool EngineControl::set_state (const XMLNode& root) { XMLNodeList clist, cclist; XMLNodeConstIterator citer, cciter; - XMLNode* child; - XMLNode* grandchild; - XMLProperty* prop = NULL; + XMLNode const * child; + XMLNode const * grandchild; if (root.name() != "AudioMIDISetup") { - return; + return false; } clist = root.children(); @@ -1229,98 +2008,60 @@ EngineControl::set_state (const XMLNode& root) continue; } - if ((prop = grandchild->property ("backend")) == 0) { - continue; - } - state->backend = prop->value (); - - if ((prop = grandchild->property ("driver")) == 0) { - continue; - } - state->driver = prop->value (); - - if ((prop = grandchild->property ("device")) == 0) { - continue; - } - state->device = prop->value (); - - if ((prop = grandchild->property ("sample-rate")) == 0) { - continue; - } - state->sample_rate = atof (prop->value ()); - - if ((prop = grandchild->property ("buffer-size")) == 0) { - continue; - } - state->buffer_size = atoi (prop->value ()); - - if ((prop = grandchild->property ("input-latency")) == 0) { - continue; - } - state->input_latency = atoi (prop->value ()); - - if ((prop = grandchild->property ("output-latency")) == 0) { - continue; - } - state->output_latency = atoi (prop->value ()); - - if ((prop = grandchild->property ("input-channels")) == 0) { - continue; - } - state->input_channels = atoi (prop->value ()); - - if ((prop = grandchild->property ("output-channels")) == 0) { + if (!grandchild->get_property ("backend", state->backend)) { continue; } - state->output_channels = atoi (prop->value ()); - if ((prop = grandchild->property ("active")) == 0) { + // If any of the required properties are not found in the state node + // then continue/skip to the next engine state + if (!grandchild->get_property ("driver", state->driver) || + !grandchild->get_property ("device", state->device) || + !grandchild->get_property ("input-device", state->input_device) || + !grandchild->get_property ("output-device", state->output_device) || + !grandchild->get_property ("sample-rate", state->sample_rate) || + !grandchild->get_property ("buffer-size", state->buffer_size) || + !grandchild->get_property ("input-latency", state->input_latency) || + !grandchild->get_property ("output-latency", state->output_latency) || + !grandchild->get_property ("input-channels", state->input_channels) || + !grandchild->get_property ("output-channels", state->output_channels) || + !grandchild->get_property ("active", state->active) || + !grandchild->get_property ("use-buffered-io", state->use_buffered_io) || + !grandchild->get_property ("midi-option", state->midi_option)) { continue; } - state->active = string_is_affirmative (prop->value ()); - if ((prop = grandchild->property ("midi-option")) == 0) { - continue; + if (!grandchild->get_property ("n-periods", state->n_periods)) { + // optional (new value in 4.5) + state->n_periods = 0; } - state->midi_option = prop->value (); state->midi_devices.clear(); XMLNode* midinode; - if ((midinode = find_named_node (*grandchild, "MIDIDevices")) != 0) { + if ((midinode = ARDOUR::find_named_node (*grandchild, "MIDIDevices")) != 0) { const XMLNodeList mnc = midinode->children(); for (XMLNodeList::const_iterator n = mnc.begin(); n != mnc.end(); ++n) { - if ((*n)->property (X_("name")) == 0 - || (*n)->property (X_("enabled")) == 0 - || (*n)->property (X_("input-latency")) == 0 - || (*n)->property (X_("output-latency")) == 0 - ) { + std::string name; + bool enabled; + uint32_t input_latency; + uint32_t output_latency; + + if (!(*n)->get_property (X_("name"), name) || + !(*n)->get_property (X_("enabled"), enabled) || + !(*n)->get_property (X_("input-latency"), input_latency) || + !(*n)->get_property (X_("output-latency"), output_latency)) { continue; } - MidiDeviceSettings ptr (new MidiDeviceSetting( - (*n)->property (X_("name"))->value (), - string_is_affirmative ((*n)->property (X_("enabled"))->value ()), - atoi ((*n)->property (X_("input-latency"))->value ()), - atoi ((*n)->property (X_("output-latency"))->value ()) - )); + MidiDeviceSettings ptr ( + new MidiDeviceSetting (name, enabled, input_latency, output_latency)); state->midi_devices.push_back (ptr); } } -#if 1 - /* remove accumulated duplicates (due to bug in ealier version) - * this can be removed again before release - */ - for (StateList::iterator i = states.begin(); i != states.end();) { - if ((*i)->backend == state->backend && - (*i)->driver == state->driver && - (*i)->device == state->device) { - i = states.erase(i); - } else { - ++i; - } + int32_t lru_val; + if (grandchild->get_property ("lru", lru_val)) { + state->lru = lru_val; } -#endif states.push_back (state); } @@ -1328,28 +2069,160 @@ EngineControl::set_state (const XMLNode& root) /* now see if there was an active state and switch the setup to it */ + /* purge states of backend that are not available in this built */ + vector backends = ARDOUR::AudioEngine::instance()->available_backends(); + vector backend_names; + + for (vector::const_iterator i = backends.begin(); i != backends.end(); ++i) { + backend_names.push_back((*i)->name); + } + for (StateList::iterator i = states.begin(); i != states.end();) { + if (std::find(backend_names.begin(), backend_names.end(), (*i)->backend) == backend_names.end()) { + i = states.erase(i); + } else { + ++i; + } + } + + states.sort (state_sort_cmp); + + /* purge old states referring to the same backend */ + const time_t now = time (NULL); + for (vector::const_iterator bi = backend_names.begin(); bi != backend_names.end(); ++bi) { + bool first = true; + for (StateList::iterator i = states.begin(); i != states.end();) { + if ((*i)->backend != *bi) { + ++i; continue; + } + // keep at latest one for every audio-system + if (first) { + first = false; + ++i; continue; + } + // also keep states used in the last 90 days. + if ((now - (*i)->lru) < 86400 * 90) { + ++i; continue; + } + assert (!(*i)->active); + i = states.erase(i); + } + } + for (StateList::const_iterator i = states.begin(); i != states.end(); ++i) { if ((*i)->active) { - ignore_changes++; - backend_combo.set_active_text ((*i)->backend); - driver_combo.set_active_text ((*i)->driver); - device_combo.set_active_text ((*i)->device); - sample_rate_combo.set_active_text (rate_as_string ((*i)->sample_rate)); - buffer_size_combo.set_active_text (bufsize_as_string ((*i)->buffer_size)); - input_latency.set_value ((*i)->input_latency); - output_latency.set_value ((*i)->output_latency); - midi_option_combo.set_active_text ((*i)->midi_option); - ignore_changes--; - break; + return set_current_state (*i); + } + } + return false; +} + +bool +EngineControl::set_current_state (const State& state) +{ + DEBUG_ECONTROL ("set_current_state"); + + boost::shared_ptr backend; + + if (!(backend = ARDOUR::AudioEngine::instance ()->set_backend ( + state->backend, downcase (std::string(PROGRAM_NAME)), ""))) { + DEBUG_ECONTROL (string_compose ("Unable to set backend to %1", state->backend)); + // this shouldn't happen as the invalid backend names should have been + // removed from the list of states. + return false; + } + + // now reflect the change in the backend in the GUI so backend_changed will + // do the right thing + backend_combo.set_active_text (state->backend); + + if (!ARDOUR::AudioEngine::instance()->setup_required ()) { + backend_changed (); + // we don't have control don't restore state + return true; + } + + + if (!state->driver.empty ()) { + if (!backend->requires_driver_selection ()) { + DEBUG_ECONTROL ("Backend should require driver selection"); + // A backend has changed from having driver selection to not having + // it or someone has been manually editing a config file and messed + // it up + return false; + } + + if (backend->set_driver (state->driver) != 0) { + DEBUG_ECONTROL (string_compose ("Unable to set driver %1", state->driver)); + // Driver names for a backend have changed and the name in the + // config file is now invalid or support for driver is no longer + // included in the backend + return false; + } + // no need to set the driver_combo as backend_changed will use + // backend->driver_name to set the active driver + } + + if (!state->device.empty ()) { + if (backend->set_device_name (state->device) != 0) { + DEBUG_ECONTROL ( + string_compose ("Unable to set device name %1", state->device)); + // device is no longer available on the system + return false; + } + // no need to set active device as it will be picked up in + // via backend_changed ()/set_device_popdown_strings + + } else { + // backend supports separate input/output devices + if (backend->set_input_device_name (state->input_device) != 0) { + DEBUG_ECONTROL (string_compose ("Unable to set input device name %1", + state->input_device)); + // input device is no longer available on the system + return false; + } + + if (backend->set_output_device_name (state->output_device) != 0) { + DEBUG_ECONTROL (string_compose ("Unable to set output device name %1", + state->input_device)); + // output device is no longer available on the system + return false; } + // no need to set active devices as it will be picked up in via + // backend_changed ()/set_*_device_popdown_strings } + + backend_changed (); + + // Now restore the state of the rest of the controls + + // We don't use a SignalBlocker as set_current_state is currently only + // called from set_state before any signals are connected. If at some point + // a more general named state mechanism is implemented and + // set_current_state is called while signals are connected then a + // SignalBlocker will need to be instantiated before setting these. + + device_combo.set_active_text (state->device); + input_device_combo.set_active_text (state->input_device); + output_device_combo.set_active_text (state->output_device); + if (0 == _desired_sample_rate && sample_rate_combo.get_sensitive ()) { + sample_rate_combo.set_active_text (rate_as_string (state->sample_rate)); + } + set_active_text_if_present (buffer_size_combo, bufsize_as_string (state->buffer_size)); + set_active_text_if_present (nperiods_combo, to_string (state->n_periods)); + input_latency.set_value (state->input_latency); + output_latency.set_value (state->output_latency); + midi_option_combo.set_active_text (state->midi_option); + use_buffered_io_button.set_active (state->use_buffered_io); + return true; } int EngineControl::push_state_to_backend (bool start) { + DEBUG_ECONTROL ("push_state_to_backend"); boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); + PBD::Unwinder protect_ignore_device_changes (ignore_device_changes, ignore_device_changes + 1); if (!backend) { return 0; @@ -1363,9 +2236,11 @@ EngineControl::push_state_to_backend (bool start) bool change_device = false; bool change_rate = false; bool change_bufsize = false; + bool change_nperiods = false; bool change_latency = false; bool change_channels = false; bool change_midi = false; + bool change_buffered_io = false; uint32_t ochan = get_output_channels (); uint32_t ichan = get_input_channels (); @@ -1382,7 +2257,20 @@ EngineControl::push_state_to_backend (bool start) } } - if (get_device_name() != backend->device_name()) { + if (backend->use_separate_input_and_output_devices()) { + if (get_input_device_name() != backend->input_device_name()) { + change_device = true; + } + if (get_output_device_name() != backend->output_device_name()) { + change_device = true; + } + } else { + if (get_device_name() != backend->device_name()) { + change_device = true; + } + } + + if (queue_device_changed) { change_device = true; } @@ -1394,10 +2282,21 @@ EngineControl::push_state_to_backend (bool start) change_bufsize = true; } + if (backend->can_set_period_size() && get_popdown_string_count (nperiods_combo) > 0 + && get_nperiods() != backend->period_size()) { + change_nperiods = true; + } + if (get_midi_option() != backend->midi_option()) { change_midi = true; } + if (backend->can_use_buffered_io()) { + if (get_use_buffered_io() != backend->get_use_buffered_io()) { + change_buffered_io = true; + } + } + /* zero-requested channels means "all available" */ if (ichan == 0) { @@ -1433,6 +2332,9 @@ EngineControl::push_state_to_backend (bool start) change_channels = true; change_latency = true; change_midi = true; + change_buffered_io = backend->can_use_buffered_io(); + change_channels = true; + change_nperiods = backend->can_set_period_size() && get_popdown_string_count (nperiods_combo) > 0; } } else { @@ -1451,6 +2353,8 @@ EngineControl::push_state_to_backend (bool start) } } + queue_device_changed = false; + if (!_have_control) { /* We do not have control over the backend, so the best we can @@ -1488,51 +2392,43 @@ EngineControl::push_state_to_backend (bool start) /* determine if we need to stop the backend before changing parameters */ - if (change_driver || change_device || change_channels || change_latency || + if (change_driver || change_device || change_channels || change_nperiods || + (change_latency && !backend->can_change_systemic_latency_when_running ()) || (change_rate && !backend->can_change_sample_rate_when_running()) || - change_midi || + change_midi || change_buffered_io || (change_bufsize && !backend->can_change_buffer_size_when_running())) { restart_required = true; } else { restart_required = false; } - if (was_running) { - - if (!change_driver && !change_device && !change_channels && !change_latency && !change_midi) { - /* no changes in any parameters that absolutely require a - * restart, so check those that might be changeable without a - * restart - */ - - if (change_rate && !backend->can_change_sample_rate_when_running()) { - /* can't do this while running ... */ - restart_required = true; - } - - if (change_bufsize && !backend->can_change_buffer_size_when_running()) { - /* can't do this while running ... */ - restart_required = true; - } - } - } if (was_running) { if (restart_required) { - if (ARDOUR_UI::instance()->disconnect_from_engine ()) { + if (ARDOUR::AudioEngine::instance()->stop()) { return -1; } } } - if (change_driver && backend->set_driver (get_driver())) { error << string_compose (_("Cannot set driver to %1"), get_driver()) << endmsg; return -1; } - if (change_device && backend->set_device_name (get_device_name())) { - error << string_compose (_("Cannot set device name to %1"), get_device_name()) << endmsg; - return -1; + if (backend->use_separate_input_and_output_devices()) { + if (change_device && backend->set_input_device_name (get_input_device_name())) { + error << string_compose (_("Cannot set input device name to %1"), get_input_device_name()) << endmsg; + return -1; + } + if (change_device && backend->set_output_device_name (get_output_device_name())) { + error << string_compose (_("Cannot set output device name to %1"), get_output_device_name()) << endmsg; + return -1; + } + } else { + if (change_device && backend->set_device_name (get_device_name())) { + error << string_compose (_("Cannot set device name to %1"), get_device_name()) << endmsg; + return -1; + } } if (change_rate && backend->set_sample_rate (get_rate())) { error << string_compose (_("Cannot set sample rate to %1"), get_rate()) << endmsg; @@ -1542,6 +2438,10 @@ EngineControl::push_state_to_backend (bool start) error << string_compose (_("Cannot set buffer size to %1"), get_buffer_size()) << endmsg; return -1; } + if (change_nperiods && backend->set_peridod_size (get_nperiods())) { + error << string_compose (_("Cannot set periods to %1"), get_nperiods()) << endmsg; + return -1; + } if (change_channels || get_input_channels() == 0 || get_output_channels() == 0) { if (backend->set_input_channels (get_input_channels())) { @@ -1568,6 +2468,10 @@ EngineControl::push_state_to_backend (bool start) backend->set_midi_option (get_midi_option()); } + if (change_buffered_io) { + backend->set_use_buffered_io (use_buffered_io_button.get_active()); + } + if (1 /* TODO */) { for (vector::const_iterator p = _midi_devices.begin(); p != _midi_devices.end(); ++p) { if (_measure_midi) { @@ -1576,6 +2480,10 @@ EngineControl::push_state_to_backend (bool start) } else { backend->set_midi_device_enabled ((*p)->name, false); } + if (backend->can_change_systemic_latency_when_running ()) { + backend->set_systemic_midi_input_latency ((*p)->name, 0); + backend->set_systemic_midi_output_latency ((*p)->name, 0); + } continue; } backend->set_midi_device_enabled ((*p)->name, (*p)->enabled); @@ -1587,7 +2495,7 @@ EngineControl::push_state_to_backend (bool start) } if (start || (was_running && restart_required)) { - if (ARDOUR_UI::instance()->reconnect_to_engine()) { + if (ARDOUR::AudioEngine::instance()->start()) { return -1; } } @@ -1604,26 +2512,30 @@ EngineControl::post_push () * necessary */ - if (_have_control) { - State state = get_saved_state_for_currently_displayed_backend_and_device (); + State state = get_saved_state_for_currently_displayed_backend_and_device (); - if (!state) { - state = save_state (); - assert (state); - } + if (!state) { + state = save_state (); + assert (state); + } else { + store_state(state); + } - /* all off */ + states.sort (state_sort_cmp); - for (StateList::iterator i = states.begin(); i != states.end(); ++i) { - (*i)->active = false; - } + /* all off */ - /* mark this one active (to be used next time the dialog is - * shown) - */ + for (StateList::iterator i = states.begin(); i != states.end(); ++i) { + (*i)->active = false; + } + + /* mark this one active (to be used next time the dialog is + * shown) + */ - state->active = true; + state->active = true; + if (_have_control) { // XXX manage_control_app_sensitivity (); } @@ -1653,27 +2565,52 @@ EngineControl::get_buffer_size () const uint32_t samples; if (sscanf (txt.c_str(), "%d", &samples) != 1) { + fprintf(stderr, "Find a trout and repeatedly slap the nearest C++ who throws exceptions without catching them.\n"); + fprintf(stderr, "Ardour will likely crash now, giving you time to get the trout.\n"); throw exception (); } return samples; } +uint32_t +EngineControl::get_nperiods () const +{ + string txt = nperiods_combo.get_active_text (); + return atoi (txt.c_str()); +} + string EngineControl::get_midi_option () const { return midi_option_combo.get_active_text(); } +bool +EngineControl::get_use_buffered_io () const +{ + return use_buffered_io_button.get_active(); +} + uint32_t EngineControl::get_input_channels() const { + if (ARDOUR::Profile->get_mixbus()) { + boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); + if (!backend) return 0; + return backend->input_channels(); + } return (uint32_t) input_channels_adjustment.get_value(); } uint32_t EngineControl::get_output_channels() const { + if (ARDOUR::Profile->get_mixbus()) { + boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); + if (!backend) return 0; + return backend->input_channels(); + } return (uint32_t) output_channels_adjustment.get_value(); } @@ -1698,7 +2635,7 @@ EngineControl::get_backend () const string EngineControl::get_driver () const { - if (driver_combo.get_sensitive() && driver_combo.get_parent()) { + if (driver_combo.get_parent()) { return driver_combo.get_active_text (); } else { return ""; @@ -1711,6 +2648,18 @@ EngineControl::get_device_name () const return device_combo.get_active_text (); } +string +EngineControl::get_input_device_name () const +{ + return input_device_combo.get_active_text (); +} + +string +EngineControl::get_output_device_name () const +{ + return output_device_combo.get_active_text (); +} + void EngineControl::control_app_button_clicked () { @@ -1723,6 +2672,58 @@ EngineControl::control_app_button_clicked () backend->launch_control_app (); } +void +EngineControl::start_stop_button_clicked () +{ + boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); + + if (!backend) { + return; + } + + if (ARDOUR::AudioEngine::instance()->running()) { + ARDOUR::AudioEngine::instance()->stop (); + } else { + if (!ARDOUR_UI::instance()->session_loaded) { + pop_splash (); + hide (); + ARDOUR::GUIIdle (); + } + start_engine (); + if (!ARDOUR_UI::instance()->session_loaded) { + ArdourDialog::on_response (RESPONSE_OK); + } + } +} + +void +EngineControl::update_devices_button_clicked () +{ + boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); + + if (!backend) { + return; + } + + if (backend->update_devices()) { + device_list_changed (); + } +} + +void +EngineControl::use_buffered_io_button_clicked () +{ + boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); + + if (!backend) { + return; + } + + bool set_buffered_io = !use_buffered_io_button.get_active(); + use_buffered_io_button.set_active (set_buffered_io); + backend->set_use_buffered_io (set_buffered_io); +} + void EngineControl::manage_control_app_sensitivity () { @@ -1745,6 +2746,10 @@ void EngineControl::set_desired_sample_rate (uint32_t sr) { _desired_sample_rate = sr; + if (ARDOUR::AudioEngine::instance ()->running () + && ARDOUR::AudioEngine::instance ()->sample_rate () != sr) { + stop_engine (); + } device_changed (); } @@ -1752,14 +2757,8 @@ void EngineControl::on_switch_page (GtkNotebookPage*, guint page_num) { if (page_num == 0) { - cancel_button->set_sensitive (true); - ok_button->set_sensitive (true); - apply_button->set_sensitive (true); _measure_midi.reset(); - } else { - cancel_button->set_sensitive (false); - ok_button->set_sensitive (false); - apply_button->set_sensitive (false); + update_sensitivity (); } if (page_num == midi_tab) { @@ -1771,8 +2770,7 @@ EngineControl::on_switch_page (GtkNotebookPage*, guint page_num) /* latency tab */ if (ARDOUR::AudioEngine::instance()->running()) { - // TODO - mark as 'stopped for latency - ARDOUR_UI::instance()->disconnect_from_engine (); + stop_engine (true); } { @@ -1828,13 +2826,20 @@ EngineControl::check_audio_latency_measurement () return true; } + if (mtdm->get_peak () > 0.707f) { + // get_peak() resets the peak-hold in the detector. + // this GUI callback is at 10Hz and so will be fine (test-signal is at higher freq) + lm_results.set_markup (string_compose (results_markup, _("Input signal is > -3dBFS. Lower the signal level (output gain, input gain) on the audio-interface."))); + return true; + } + if (mtdm->err () > 0.3) { mtdm->invert (); mtdm->resolve (); } char buf[256]; - ARDOUR::framecnt_t const sample_rate = ARDOUR::AudioEngine::instance()->sample_rate(); + ARDOUR::samplecnt_t const sample_rate = ARDOUR::AudioEngine::instance()->sample_rate(); if (sample_rate == 0) { lm_results.set_markup (string_compose (results_markup, _("Disconnected from audio engine"))); @@ -1842,12 +2847,12 @@ EngineControl::check_audio_latency_measurement () return false; } - int frames_total = mtdm->del(); - int extra = frames_total - ARDOUR::AudioEngine::instance()->latency_signal_delay(); + int samples_total = mtdm->del(); + int extra = samples_total - ARDOUR::AudioEngine::instance()->latency_signal_delay(); snprintf (buf, sizeof (buf), "%s%d samples (%.3lf ms)\n%s%d samples (%.3lf ms)", _("Detected roundtrip latency: "), - frames_total, frames_total * 1000.0f/sample_rate, + samples_total, samples_total * 1000.0f/sample_rate, _("Systemic latency: "), extra, extra * 1000.0f/sample_rate); @@ -1865,14 +2870,15 @@ EngineControl::check_audio_latency_measurement () solid = false; } + lm_results.set_markup (string_compose (results_markup, buf)); + if (solid) { + have_lm_results = true; end_latency_detection (); lm_use_button.set_sensitive (true); - have_lm_results = true; + return false; } - lm_results.set_markup (string_compose (results_markup, buf)); - return true; } @@ -1887,7 +2893,7 @@ EngineControl::check_midi_latency_measurement () } char buf[256]; - ARDOUR::framecnt_t const sample_rate = ARDOUR::AudioEngine::instance()->sample_rate(); + ARDOUR::samplecnt_t const sample_rate = ARDOUR::AudioEngine::instance()->sample_rate(); if (sample_rate == 0) { lm_results.set_markup (string_compose (results_markup, _("Disconnected from audio engine"))); @@ -1895,11 +2901,11 @@ EngineControl::check_midi_latency_measurement () return false; } - ARDOUR::framecnt_t frames_total = mididm->latency(); - ARDOUR::framecnt_t extra = frames_total - ARDOUR::AudioEngine::instance()->latency_signal_delay(); + ARDOUR::samplecnt_t samples_total = mididm->latency(); + ARDOUR::samplecnt_t extra = samples_total - ARDOUR::AudioEngine::instance()->latency_signal_delay(); snprintf (buf, sizeof (buf), "%s%" PRId64" samples (%.1lf ms) dev: %.2f[spl]\n%s%" PRId64" samples (%.1lf ms)", _("Detected roundtrip latency: "), - frames_total, frames_total * 1000.0f / sample_rate, mididm->deviation (), + samples_total, samples_total * 1000.0f / sample_rate, mididm->deviation (), _("Systemic latency: "), extra, extra * 1000.0f / sample_rate); @@ -1921,9 +2927,10 @@ EngineControl::check_midi_latency_measurement () } if (solid) { + have_lm_results = true; end_latency_detection (); lm_use_button.set_sensitive (true); - have_lm_results = true; + lm_results.set_markup (string_compose (results_markup, buf)); return false; } else if (mididm->processed () > 400) { have_lm_results = false; @@ -1957,13 +2964,6 @@ EngineControl::start_latency_detection () lm_output_channel_combo.set_sensitive (false); lm_running = true; } - - lm_back_button_signal.disconnect(); - if (_measure_midi) { - lm_back_button_signal = lm_back_button.signal_clicked().connect (sigc::bind (sigc::mem_fun (notebook, &Gtk::Notebook::set_current_page), midi_tab)); - } else { - lm_back_button_signal = lm_back_button.signal_clicked().connect (sigc::bind (sigc::mem_fun (notebook, &Gtk::Notebook::set_current_page), 0)); - } } void @@ -1973,8 +2973,6 @@ EngineControl::end_latency_detection () ARDOUR::AudioEngine::instance()->stop_latency_detection (); lm_measure_label.set_text (_("Measure")); if (!have_lm_results) { - lm_results.set_markup (string_compose (results_markup, _("No measurement results yet"))); - } else { lm_use_button.set_sensitive (false); } lm_input_channel_combo.set_sensitive (true); @@ -1992,19 +2990,31 @@ EngineControl::latency_button_clicked () } } +void +EngineControl::latency_back_button_clicked () +{ + ARDOUR::AudioEngine::instance()->stop_latency_detection (); + notebook.set_current_page(0); +} + void EngineControl::use_latency_button_clicked () { + boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); if (_measure_midi) { ARDOUR::MIDIDM* mididm = ARDOUR::AudioEngine::instance()->mididm (); if (!mididm) { return; } - ARDOUR::framecnt_t frames_total = mididm->latency(); - ARDOUR::framecnt_t extra = frames_total - ARDOUR::AudioEngine::instance()->latency_signal_delay(); - uint32_t one_way = max ((ARDOUR::framecnt_t) 0, extra / 2); + ARDOUR::samplecnt_t samples_total = mididm->latency(); + ARDOUR::samplecnt_t extra = samples_total - ARDOUR::AudioEngine::instance()->latency_signal_delay(); + uint32_t one_way = max ((ARDOUR::samplecnt_t) 0, extra / 2); _measure_midi->input_latency = one_way; _measure_midi->output_latency = one_way; + if (backend->can_change_systemic_latency_when_running ()) { + backend->set_systemic_midi_input_latency (_measure_midi->name, one_way); + backend->set_systemic_midi_output_latency (_measure_midi->name, one_way); + } notebook.set_current_page (midi_tab); } else { MTDM* mtdm = ARDOUR::AudioEngine::instance()->mtdm (); @@ -2018,12 +3028,15 @@ EngineControl::use_latency_button_clicked () input_latency_adjustment.set_value (one_way); output_latency_adjustment.set_value (one_way); + if (backend->can_change_systemic_latency_when_running ()) { + backend->set_systemic_input_latency (one_way); + backend->set_systemic_output_latency (one_way); + } /* back to settings page */ notebook.set_current_page (0); -} } - +} bool EngineControl::on_delete_event (GdkEventAny* ev) @@ -2041,15 +3054,23 @@ EngineControl::engine_running () boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); assert (backend); - buffer_size_combo.set_active_text (bufsize_as_string (backend->buffer_size())); + set_active_text_if_present (buffer_size_combo, bufsize_as_string (backend->buffer_size())); sample_rate_combo.set_active_text (rate_as_string (backend->sample_rate())); - buffer_size_combo.set_sensitive (true); - sample_rate_combo.set_sensitive (true); + if (backend->can_set_period_size ()) { + set_active_text_if_present (nperiods_combo, to_string (backend->period_size())); + } connect_disconnect_button.set_label (string_compose (_("Disconnect from %1"), backend->name())); + connect_disconnect_button.show(); started_at_least_once = true; + if (_have_control) { + engine_status.set_markup(string_compose ("%1", _("Running"))); + } else { + engine_status.set_markup(string_compose ("%1", _("Connected"))); + } + update_sensitivity(); } void @@ -2058,20 +3079,44 @@ EngineControl::engine_stopped () boost::shared_ptr backend = ARDOUR::AudioEngine::instance()->current_backend(); assert (backend); - buffer_size_combo.set_sensitive (false); connect_disconnect_button.set_label (string_compose (_("Connect to %1"), backend->name())); + connect_disconnect_button.show(); + + if (_have_control) { + engine_status.set_markup(string_compose ("%1", _("Stopped"))); + } else { + engine_status.set_markup(X_("")); + } + + update_sensitivity(); +} - sample_rate_combo.set_sensitive (true); - buffer_size_combo.set_sensitive (true); +void +EngineControl::device_list_changed () +{ + if (ignore_device_changes) { + return; + } + PBD::Unwinder protect_ignore_changes (ignore_changes, ignore_changes + 1); // ?? + list_devices (); + midi_option_changed(); } void EngineControl::connect_disconnect_click() { if (ARDOUR::AudioEngine::instance()->running()) { - ARDOUR_UI::instance()->disconnect_from_engine (); + stop_engine (); } else { - ARDOUR_UI::instance()->reconnect_to_engine (); + if (!ARDOUR_UI::instance()->session_loaded) { + pop_splash (); + hide (); + ARDOUR::GUIIdle (); + } + start_engine (); + if (!ARDOUR_UI::instance()->session_loaded) { + ArdourDialog::on_response (RESPONSE_OK); + } } }