13
0

Autostart must {try to} use the session sample-rate. There is no ambiguity here.

This commit is contained in:
Ben Loftis 2019-12-06 13:25:44 -06:00
parent 941aa20148
commit 7457fd2e27

View File

@ -426,21 +426,9 @@ StartupFSM::start_audio_midi_setup ()
if (setup_required) {
/* Note: if autostart is enabled, and starting the engine is
* successful, but the session SR differs, there will be no
* chance to reset it.
*
* We could change this trivially with a call to
* AudioEngine::set_sample_rate(), but that opens a can of
* worms about policy, UX, GUI and more, because it isn't clear
* whether that's the correct thing to do. So for now (Nov
* 2019) we simply try the autostart if the user asked for it,
* and if necessary a session SR mismatch dialog will appear
* during loading.
*/
if (!session_is_new && (Config->get_try_autostart_engine () || g_getenv ("ARDOUR_TRY_AUTOSTART_ENGINE"))) {
AudioEngine::instance()->set_sample_rate(session_existing_sample_rate);
if (!AudioEngine::instance()->start ()) {
if (ARDOUR::AudioEngine::instance()->running()) {
DEBUG_TRACE (DEBUG::GuiStartup, "autostart successful, audio/MIDI setup dialog not required\n");