ardour/livetrax
13
0
Fork 0
Code

Compare the string returned by backend->driver_name() against driver names in EngineControl

Browse Source 
Don't assume an empty string has any meaning and compare string returned with
enumerated drivers
This commit is contained in:
Tim Mayberry 2015-08-10 21:44:29 +10:00
parent e20d74079a
commit 495dd2e52a
1 changed files with 7 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
gtk2_ardour/engine_dialog.cc
View File

@ -915,27 +915,24 @@ EngineControl::set_driver_popdown_strings ()
boost::shared_ptr<ARDOUR::AudioBackend> backend = ARDOUR::AudioEngine::instance()->current_backend();
vector<string> drivers = backend->enumerate_drivers();
if (drivers.empty()) {
if (drivers.empty ()) {
// This is an error...?
return false;
}
string current_driver;
current_driver = backend->driver_name ();
string current_driver = backend->driver_name ();
DEBUG_ECONTROL (string_compose ("backend->driver_name: %1", current_driver));
// driver might not have been set yet
if (current_driver == "") {
current_driver = driver_combo.get_active_text ();
if (current_driver == "")
// driver has never been set, make sure it's not blank
current_driver = drivers.front ();
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));
string_compose ("driver_combo.set_active_text: %1", current_driver));
driver_combo.set_active_text (current_driver);
return true;
}