Make global static children variable an xml node object variable.
[ardour.git] / gtk2_ardour / engine_dialog.cc
index 602e6224c8343c2d10447f4042d40cbfa5c4843d..65e3b547a31d1061ebccc983d7f839edf16486ad 100644 (file)
@@ -1,13 +1,20 @@
 #include <vector>
 #include <cmath>
 #include <fstream>
+#include <map>
 
 #include <glibmm.h>
+#include <gtkmm/messagedialog.h>
+#include <pbd/xml++.h>
 
 #ifdef __APPLE__
 #include <CoreAudio/CoreAudio.h>
 #include <CoreFoundation/CFString.h>
-#endif __APPLE__
+#include <sys/param.h>
+#include <mach-o/dyld.h>
+#else
+#include <alsa/asoundlib.h>
+#endif
 
 #include <ardour/profile.h>
 #include <jack/jack.h>
 
 #include <pbd/convert.h>
 #include <pbd/error.h>
+#include <pbd/pathscanner.h>
+
+#ifdef __APPLE
+#include <CFBundle.h>
+#endif
 
 #include "engine_dialog.h"
 #include "i18n.h"
@@ -45,13 +57,22 @@ EngineControl::EngineControl ()
          verbose_output_button (_("Verbose output")),
          start_button (_("Start")),
          stop_button (_("Stop")),
+#ifdef __APPLE__
+         basic_packer (5, 2),
+         options_packer (4, 2),
+         device_packer (4, 2)
+#else
          basic_packer (8, 2),
-         options_packer (12, 2),
-         device_packer (3, 2)
+         options_packer (14, 2),
+         device_packer (6, 2)
+#endif   
 {
        using namespace Notebook_Helpers;
        Label* label;
        vector<string> strings;
+       int row = 0;
+
+       _used = false;
 
        strings.push_back (_("8000Hz"));
        strings.push_back (_("22050Hz"));
@@ -76,74 +97,100 @@ EngineControl::EngineControl ()
        set_popdown_strings (period_size_combo, strings);
        period_size_combo.set_active_text ("1024");
 
+       strings.clear ();
+       strings.push_back (_("None"));
+       strings.push_back (_("Triangular"));
+       strings.push_back (_("Rectangular"));
+       strings.push_back (_("Shaped"));
+       set_popdown_strings (dither_mode_combo, strings);
+       dither_mode_combo.set_active_text (_("None"));
+
        /* basic parameters */
 
        basic_packer.set_spacings (6);
 
        strings.clear ();
-#ifndef __APPLE__
+#ifdef __APPLE__
+       strings.push_back (X_("CoreAudio"));
+#else
        strings.push_back (X_("ALSA"));
        strings.push_back (X_("OSS"));
        strings.push_back (X_("FFADO"));
-#else
-       strings.push_back (X_("CoreAudio"));
 #endif
        strings.push_back (X_("NetJACK"));
        strings.push_back (X_("Dummy"));
        set_popdown_strings (driver_combo, strings);
        driver_combo.set_active_text (strings.front());
 
-       /* figure out available devices and set up interface_combo */
-
-       enumerate_devices ();
        driver_combo.signal_changed().connect (mem_fun (*this, &EngineControl::driver_changed));
        driver_changed ();
 
        strings.clear ();
-       strings.push_back (_("Duplex"));
+       strings.push_back (_("Playback/Recording on 1 Device"));
+       strings.push_back (_("Playback/Recording on 2 Devices"));
        strings.push_back (_("Playback only"));
-       strings.push_back (_("Capture only"));
+       strings.push_back (_("Recording only"));
        set_popdown_strings (audio_mode_combo, strings);
        audio_mode_combo.set_active_text (strings.front());
 
        audio_mode_combo.signal_changed().connect (mem_fun (*this, &EngineControl::audio_mode_changed));
        audio_mode_changed ();
 
+       row = 0;
+
        label = manage (new Label (_("Driver")));
-       basic_packer.attach (*label, 0, 1, 0, 1, FILL|EXPAND, (AttachOptions) 0);
-       basic_packer.attach (driver_combo, 1, 2, 0, 1, FILL|EXPAND, (AttachOptions) 0);
+       basic_packer.attach (*label, 0, 1, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       basic_packer.attach (driver_combo, 1, 2, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       row++;
 
        label = manage (new Label (_("Interface")));
-       basic_packer.attach (*label, 0, 1, 1, 2, FILL|EXPAND, (AttachOptions) 0);
-       basic_packer.attach (interface_combo, 1, 2, 1, 2, FILL|EXPAND, (AttachOptions) 0);
+       basic_packer.attach (*label, 0, 1, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       basic_packer.attach (interface_combo, 1, 2, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       row++;
 
        label = manage (new Label (_("Sample Rate")));
-       basic_packer.attach (*label, 0, 1, 2, 3, FILL|EXPAND, (AttachOptions) 0);
-       basic_packer.attach (sample_rate_combo, 1, 2, 2, 3, FILL|EXPAND, (AttachOptions) 0);
+       basic_packer.attach (*label, 0, 1, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       basic_packer.attach (sample_rate_combo, 1, 2, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       row++;
 
        label = manage (new Label (_("Buffer size")));
-       basic_packer.attach (*label, 0, 1, 3, 4, FILL|EXPAND, (AttachOptions) 0);
-       basic_packer.attach (period_size_combo, 1, 2, 3, 4, FILL|EXPAND, (AttachOptions) 0);
+       basic_packer.attach (*label, 0, 1, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       basic_packer.attach (period_size_combo, 1, 2, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       row++;
 
+#ifndef __APPLE__
        label = manage (new Label (_("Number of buffers")));
-       basic_packer.attach (*label, 0, 1, 4, 5, FILL|EXPAND, (AttachOptions) 0);
-       basic_packer.attach (periods_spinner, 1, 2, 4, 5, FILL|EXPAND, (AttachOptions) 0);
+       basic_packer.attach (*label, 0, 1, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       basic_packer.attach (periods_spinner, 1, 2, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
        periods_spinner.set_value (2);
+       row++;
+#endif
 
        label = manage (new Label (_("Approximate latency")));
-       basic_packer.attach (*label, 0, 1, 5, 6, FILL|EXPAND, (AttachOptions) 0);
-       basic_packer.attach (latency_label, 1, 2, 5, 6, FILL|EXPAND, (AttachOptions) 0);
+       label->set_alignment (0.0, 0.5);
+       basic_packer.attach (*label, 0, 1, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       basic_packer.attach (latency_label, 1, 2, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       row++;
 
        sample_rate_combo.signal_changed().connect (mem_fun (*this, &EngineControl::redisplay_latency));
        periods_adjustment.signal_value_changed().connect (mem_fun (*this, &EngineControl::redisplay_latency));
        period_size_combo.signal_changed().connect (mem_fun (*this, &EngineControl::redisplay_latency));
        redisplay_latency();
+       row++;
+       /* no audio mode with CoreAudio, its duplex or nuthin' */
 
+#ifndef __APPLE__
        label = manage (new Label (_("Audio Mode")));
-       basic_packer.attach (*label, 0, 1, 6, 7, FILL|EXPAND, (AttachOptions) 0);
-       basic_packer.attach (audio_mode_combo, 1, 2, 6, 7, FILL|EXPAND, (AttachOptions) 0);
+       basic_packer.attach (*label, 0, 1, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       basic_packer.attach (audio_mode_combo, 1, 2, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       row++;
+#endif
+
+       interface_combo.set_size_request (125, -1);
+       input_device_combo.set_size_request (125, -1);
+       output_device_combo.set_size_request (125, -1);
 
-       /* 
+       /*
 
        if (engine_running()) {
                start_button.set_sensitive (false);
@@ -162,26 +209,42 @@ EngineControl::EngineControl ()
 
        /* options */
 
-       options_packer.attach (realtime_button, 0, 1, 0, 1, FILL|EXPAND, (AttachOptions) 0);
-       label = manage (new Label (_("Realtime Priority")));
-       options_packer.attach (*label, 0, 1, 1, 2, FILL|EXPAND, (AttachOptions) 0);
-       options_packer.attach (priority_spinner, 1, 2, 1, 2, FILL|EXPAND, (AttachOptions) 0);
-       priority_spinner.set_value (60);
+       options_packer.set_spacings (6);
+       row = 0;
+
+       options_packer.attach (realtime_button, 1, 2, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       ++row;
 
        realtime_button.signal_toggled().connect (mem_fun (*this, &EngineControl::realtime_changed));
        realtime_changed ();
 
 #ifndef __APPLE__
-       options_packer.attach (no_memory_lock_button, 0, 1, 2, 3, FILL|EXPAND, (AttachOptions) 0);
-       options_packer.attach (unlock_memory_button, 0, 1, 3, 4, FILL|EXPAND, (AttachOptions) 0);
-       options_packer.attach (soft_mode_button, 0, 1, 4, 5, FILL|EXPAND, (AttachOptions) 0);
-       options_packer.attach (monitor_button, 0, 1, 5, 6, FILL|EXPAND, (AttachOptions) 0);
-       options_packer.attach (force16bit_button, 0, 1, 6, 7, FILL|EXPAND, (AttachOptions) 0);
-       options_packer.attach (hw_monitor_button, 0, 1, 7, 8, FILL|EXPAND, (AttachOptions) 0);
-       options_packer.attach (hw_meter_button, 0, 1, 8, 9, FILL|EXPAND, (AttachOptions) 0);
-       options_packer.attach (verbose_output_button, 0, 1, 9, 10, FILL|EXPAND, (AttachOptions) 0);
+       label = manage (new Label (_("Realtime Priority")));
+       label->set_alignment (1.0, 0.5);
+       options_packer.attach (*label, 0, 1, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       options_packer.attach (priority_spinner, 1, 2, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       ++row;
+       priority_spinner.set_value (60);
+
+       options_packer.attach (no_memory_lock_button, 1, 2, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       ++row;
+       options_packer.attach (unlock_memory_button, 1, 2, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       ++row;
+       options_packer.attach (soft_mode_button, 1, 2, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       ++row;
+       options_packer.attach (monitor_button, 1, 2, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       ++row;
+       options_packer.attach (force16bit_button, 1, 2, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       ++row;
+       options_packer.attach (hw_monitor_button, 1, 2, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       ++row;
+       options_packer.attach (hw_meter_button, 1, 2, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       ++row;
+       options_packer.attach (verbose_output_button, 1, 2, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       ++row;
 #else 
-       options_packer.attach (verbose_output_button, 0, 1, 2, 3, FILL|EXPAND, (AttachOptions) 0);
+       options_packer.attach (verbose_output_button, 1, 2, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       ++row;
 #endif
 
        strings.clear ();
@@ -194,61 +257,95 @@ EngineControl::EngineControl ()
        timeout_combo.set_active_text (strings.front ());
 
        label = manage (new Label (_("Client timeout")));
-       options_packer.attach (*label, 0, 1, 11, 12, (AttachOptions) 0, (AttachOptions) 0);
-       options_packer.attach (timeout_combo, 1, 2, 11, 12, FILL|EXPAND, AttachOptions(0));
+       label->set_alignment (1.0, 0.5);
+       options_packer.attach (timeout_combo, 1, 2, row, row + 1, FILL|EXPAND, AttachOptions(0));
+       options_packer.attach (*label, 0, 1, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       ++row;
 
        label = manage (new Label (_("Number of ports")));
-       options_packer.attach (*label, 0, 1, 12, 13, (AttachOptions) 0, (AttachOptions) 0);
-       options_packer.attach (ports_spinner, 1, 2, 12, 13, FILL|EXPAND, AttachOptions(0));
+       label->set_alignment (1.0, 0.5);
+       options_packer.attach (ports_spinner, 1, 2, row, row + 1, FILL|EXPAND, AttachOptions(0));
+       options_packer.attach (*label, 0, 1, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       ++row;
 
-       strings.clear ();
+#ifndef __APPLE__
+       label = manage (new Label (_("Dither")));       
+       label->set_alignment (1.0, 0.5);
+       options_packer.attach (dither_mode_combo, 1, 2, row, row + 1, FILL|EXPAND, AttachOptions(0));
+       options_packer.attach (*label, 0, 1, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+       ++row;
+#endif
 
-       find_jack_servers (strings);
+       find_jack_servers (server_strings);
 
-       if (strings.empty()) {
+       if (server_strings.empty()) {
                fatal << _("No JACK server found anywhere on this system. Please install JACK and restart") << endmsg;
                /*NOTREACHED*/
        }
        
-       set_popdown_strings (serverpath_combo, strings);
-       serverpath_combo.set_active_text (strings.front());
+       set_popdown_strings (serverpath_combo, server_strings);
+       serverpath_combo.set_active_text (server_strings.front());
 
-       if (strings.size() > 1) {
+       if (server_strings.size() > 1) {
                label = manage (new Label (_("Server:")));
-               options_packer.attach (*label, 0, 1, 11, 12, (AttachOptions) 0, (AttachOptions) 0);
-               options_packer.attach (serverpath_combo, 1, 2, 11, 12, FILL|EXPAND, (AttachOptions) 0);
+               options_packer.attach (*label, 0, 1, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+               label->set_alignment (0.0, 0.5);
+               options_packer.attach (serverpath_combo, 1, 2, row, row + 1, FILL|EXPAND, (AttachOptions) 0);
+               ++row;
        }
 
        /* device settings */
 
        device_packer.set_spacings (6);
+       row = 0;
 
+#ifndef __APPLE__
        label = manage (new Label (_("Input device")));
-       device_packer.attach (*label, 0, 1, 0, 1, FILL|EXPAND, (AttachOptions) 0);
-       device_packer.attach (input_device_combo, 1, 2, 0, 1, FILL|EXPAND, (AttachOptions) 0);
+       label->set_alignment (1.0, 0.5);
+       device_packer.attach (*label, 0, 1, row, row+1, FILL|EXPAND, (AttachOptions) 0);
+       device_packer.attach (input_device_combo, 1, 2, row, row+1, FILL|EXPAND, (AttachOptions) 0);
+       ++row;
        label = manage (new Label (_("Output device")));
-       device_packer.attach (*label, 0, 1, 1, 2, FILL|EXPAND, (AttachOptions) 0);
-       device_packer.attach (output_device_combo, 1, 2, 1, 2, FILL|EXPAND, (AttachOptions) 0); 
+       label->set_alignment (1.0, 0.5);
+       device_packer.attach (*label, 0, 1, row, row+1, FILL|EXPAND, (AttachOptions) 0);
+       device_packer.attach (output_device_combo, 1, 2, row, row+1, FILL|EXPAND, (AttachOptions) 0);   
+       ++row;
+#endif
        label = manage (new Label (_("Input channels")));
-       device_packer.attach (*label, 0, 1, 2, 3, FILL|EXPAND, (AttachOptions) 0);
-       device_packer.attach (input_channels, 1, 2, 2, 3, FILL|EXPAND, (AttachOptions) 0);
+       label->set_alignment (1.0, 0.5);
+       device_packer.attach (*label, 0, 1, row, row+1, FILL|EXPAND, (AttachOptions) 0);
+       device_packer.attach (input_channels, 1, 2, row, row+1, FILL|EXPAND, (AttachOptions) 0);
+       ++row;
        label = manage (new Label (_("Output channels")));
-       device_packer.attach (*label, 0, 1, 3, 4, FILL|EXPAND, (AttachOptions) 0);
-       device_packer.attach (output_channels, 1, 2, 3, 4, FILL|EXPAND, (AttachOptions) 0);
-       label = manage (new Label (_("Input latency (samples)")));
-       device_packer.attach (*label, 0, 1, 4, 5, FILL|EXPAND, (AttachOptions) 0);
-       device_packer.attach (input_latency, 1, 2, 4, 5, FILL|EXPAND, (AttachOptions) 0);
-       label = manage (new Label (_("Output latency (samples)")));
-       device_packer.attach (*label, 0, 1, 5, 6, FILL|EXPAND, (AttachOptions) 0);
-       device_packer.attach (output_latency, 1, 2, 5, 6, FILL|EXPAND, (AttachOptions) 0);
-
-       notebook.pages().push_back (TabElem (basic_packer, _("Basics")));
-       notebook.pages().push_back (TabElem (options_packer, _("Options")));
-       notebook.pages().push_back (TabElem (device_packer, _("Device Parameters")));
+       label->set_alignment (1.0, 0.5);
+       device_packer.attach (*label, 0, 1, row, row+1, FILL|EXPAND, (AttachOptions) 0);
+       device_packer.attach (output_channels, 1, 2, row, row+1, FILL|EXPAND, (AttachOptions) 0);
+       ++row;
+       label = manage (new Label (_("Hardware input latency (samples)")));
+       label->set_alignment (1.0, 0.5);
+       device_packer.attach (*label, 0, 1, row, row+1, FILL|EXPAND, (AttachOptions) 0);
+       device_packer.attach (input_latency, 1, 2, row, row+1, FILL|EXPAND, (AttachOptions) 0);
+       ++row;
+       label = manage (new Label (_("Hardware output latency (samples)")));
+       label->set_alignment (1.0, 0.5);
+       device_packer.attach (*label, 0, 1, row, row+1, FILL|EXPAND, (AttachOptions) 0);
+       device_packer.attach (output_latency, 1, 2, row, row+1, FILL|EXPAND, (AttachOptions) 0);
+       ++row;
+
+       basic_hbox.pack_start (basic_packer, false, false);
+       options_hbox.pack_start (options_packer, false, false);
+
+       device_packer.set_border_width (12);
+       options_packer.set_border_width (12);
+       basic_packer.set_border_width (12);
+
+       notebook.pages().push_back (TabElem (basic_hbox, _("Device")));
+       notebook.pages().push_back (TabElem (options_hbox, _("Options")));
+       notebook.pages().push_back (TabElem (device_packer, _("Advanced")));
+       notebook.set_border_width (12);
 
        set_border_width (12);
        pack_start (notebook);
-
 }
 
 EngineControl::~EngineControl ()
@@ -260,11 +357,13 @@ void
 EngineControl::build_command_line (vector<string>& cmd)
 {
        string str;
+       string driver;
        bool using_oss = false;
        bool using_alsa = false;
        bool using_coreaudio = false;
        bool using_netjack = false;
        bool using_ffado = false;
+       bool using_dummy = false;
 
        /* first, path to jackd */
 
@@ -311,37 +410,72 @@ EngineControl::build_command_line (vector<string>& cmd)
 
        cmd.push_back ("-d");
 
-       str = driver_combo.get_active_text ();
-       if (str == X_("ALSA")) {
+       driver = driver_combo.get_active_text ();
+       if (driver == X_("ALSA")) {
                using_alsa = true;
                cmd.push_back ("alsa");
-       } else if (str == X_("OSS")) {
+       } else if (driver == X_("OSS")) {
                using_oss = true;
                cmd.push_back ("oss");
-       } else if (str == X_("CoreAudio")) {
+       } else if (driver == X_("CoreAudio")) {
                using_coreaudio = true;
                cmd.push_back ("coreaudio");
-       } else if (str == X_("NetJACK")) {
+       } else if (driver == X_("NetJACK")) {
                using_netjack = true;
                cmd.push_back ("netjack");
-       } else if (str == X_("FFADO")) {
+       } else if (driver == X_("FFADO")) {
                using_ffado = true;
-               cmd.push_back ("ffado");
+
+               /* do this until FFADO becomes the standard */
+
+               char* hack = getenv ("ARDOUR_FIREWIRE_DRIVER_NAME");
+
+               if (hack) {
+                       cmd.push_back (hack);
+               } else {
+                       cmd.push_back ("freebob");
+               }
+
+       } else if ( driver == X_("Dummy")) {
+               using_dummy = true;
+               cmd.push_back ("dummy");
        }
 
        /* driver arguments */
 
-       str = audio_mode_combo.get_active_text();
-       if (str == _("Duplex")) {
-               /* relax */
-       } else if (str == _("Playback only")) {
-               cmd.push_back ("-P");
-       } else if (str == _("Capture only")) {
-               cmd.push_back ("-C");
-       }
+       if (!using_coreaudio) {
+               str = audio_mode_combo.get_active_text();
+               
+               if (str == _("Playback/Recording on 1 Device")) {
+                       
+                       /* relax */
+                       
+               } else if (str == _("Playback/Recording on 2 Devices")) {
+                       
+                       string input_device = get_device_name (driver, input_device_combo.get_active_text());
+                       string output_device = get_device_name (driver, output_device_combo.get_active_text());
+
+                       if (input_device.empty() || output_device.empty()) {
+                               cmd.clear ();
+                               return;
+                       }
+
+                       cmd.push_back ("-C");
+                       cmd.push_back (input_device);
+                       cmd.push_back ("-P");
+                       cmd.push_back (output_device);
 
-       cmd.push_back ("-n");
-       cmd.push_back (to_string ((uint32_t) floor (periods_spinner.get_value()), std::dec));
+               } else if (str == _("Playback only")) {
+                       cmd.push_back ("-P");
+               } else if (str == _("Recording only")) {
+                       cmd.push_back ("-C");
+               }
+
+               if (! using_dummy ) {
+                       cmd.push_back ("-n");
+                       cmd.push_back (to_string ((uint32_t) floor (periods_spinner.get_value()), std::dec));
+               }
+       }
 
        cmd.push_back ("-r");
        cmd.push_back (to_string (get_rate(), std::dec));
@@ -350,9 +484,18 @@ EngineControl::build_command_line (vector<string>& cmd)
        cmd.push_back (period_size_combo.get_active_text());
 
        if (using_alsa) {
+               
+               if (audio_mode_combo.get_active_text() != _("Playback/Recording on 2 Devices")) {
 
-               cmd.push_back ("-d");
-               cmd.push_back (interface_combo.get_active_text());
+                       string device = get_device_name (driver, interface_combo.get_active_text());
+                       if (device.empty()) {
+                               cmd.clear ();
+                               return;
+                       }
+
+                       cmd.push_back ("-d");
+                       cmd.push_back (device);
+               } 
 
                if (hw_meter_button.get_active()) {
                        cmd.push_back ("-M");
@@ -363,6 +506,7 @@ EngineControl::build_command_line (vector<string>& cmd)
                }
 
                str = dither_mode_combo.get_active_text();
+
                if (str == _("None")) {
                } else if (str == _("Triangular")) {
                        cmd.push_back ("-z triangular");
@@ -382,8 +526,18 @@ EngineControl::build_command_line (vector<string>& cmd)
 
        } else if (using_coreaudio) {
 
-               cmd.push_back ("-I");
-               cmd.push_back (interface_combo.get_active_text());
+#ifdef __APPLE__
+               // note: older versions of the CoreAudio JACK backend use -n instead of -d here
+               
+               string device = get_device_name (driver, interface_combo.get_active_text());
+               if (device.empty()) {
+                       cmd.clear ();
+                       return;
+               }
+
+               cmd.push_back ("-d");
+               cmd.push_back (device);
+#endif
 
        } else if (using_oss) {
 
@@ -406,13 +560,16 @@ EngineControl::engine_running ()
 }
 
 int
-EngineControl::start_engine ()
+EngineControl::setup_engine ()
 {
        vector<string> args;
        std::string cwd = "/tmp";
-       int ret = 0;
 
        build_command_line (args);
+       
+       if (args.empty()) {
+               return 1; // try again
+       }
 
        Glib::ustring jackdrc_path = Glib::get_home_dir();
        jackdrc_path += "/.jackdrc";
@@ -422,61 +579,69 @@ EngineControl::start_engine ()
                error << string_compose (_("cannot open JACK rc file %1 to store parameters"), jackdrc_path) << endmsg;
                return -1;
        }
-
+       cerr << "JACK COMMAND: ";
        for (vector<string>::iterator i = args.begin(); i != args.end(); ++i) {
+               cerr << (*i) << ' ';
                jackdrc << (*i) << ' ';
        }
+       cerr << endl;
        jackdrc << endl;
        jackdrc.close ();
-       
-#if 0
 
-       try {
-               spawn_async_with_pipes (cwd, args, SpawnFlags (0), sigc::slot<void>(), &engine_pid, &engine_stdin, &engine_stdout, &engine_stderr);
-       }
-       
-       catch (Glib::Exception& err) {
-               error << _("could not start JACK server: ") << err.what() << endmsg;
-               ret = -1;
-       }
-#endif
+       _used = true;
 
-       return ret;
-}
-
-int
-EngineControl::stop_engine ()
-{
-       close (engine_stdin);
-       close (engine_stderr);
-       close (engine_stdout);
-       spawn_close_pid (engine_pid);
        return 0;
 }
 
 void
 EngineControl::realtime_changed ()
 {
+#ifndef __APPLE__
        priority_spinner.set_sensitive (realtime_button.get_active());
+#endif
 }
 
 void
-EngineControl::enumerate_devices ()
+EngineControl::enumerate_devices (const string& driver)
 {
        /* note: case matters for the map keys */
 
-#ifdef __APPLE__
-       devices["CoreAudio"] = enumerate_coreaudio_devices ();
+       if (driver == "CoreAudio") {
+#ifdef __APPLE__               
+               devices[driver] = enumerate_coreaudio_devices ();
+#endif
+
+#ifndef __APPLE__
+       } else if (driver == "ALSA") {
+               devices[driver] = enumerate_alsa_devices ();
+       } else if (driver == "FFADO") {
+               devices[driver] = enumerate_ffado_devices ();
+       } else if (driver == "OSS") {
+               devices[driver] = enumerate_oss_devices ();
+       } else if (driver == "Dummy") {
+               devices[driver] = enumerate_dummy_devices ();
+       } else if (driver == "NetJACK") {
+               devices[driver] = enumerate_netjack_devices ();
+       }
 #else
-       devices["ALSA"] = enumerate_alsa_devices ();
-       devices["FFADO"] = enumerate_ffado_devices ();
-       devices["OSS"] = enumerate_oss_devices ();
-       devices["Dummy"] = enumerate_dummy_devices ();
-       devices["NetJACK"] = enumerate_netjack_devices ();
+        }
 #endif
 }
 
 #ifdef __APPLE__
+static OSStatus 
+getDeviceUIDFromID( AudioDeviceID id, char *name, size_t nsize)
+{
+       UInt32 size = sizeof(CFStringRef);
+       CFStringRef UI;
+       OSStatus res = AudioDeviceGetProperty(id, 0, false,
+               kAudioDevicePropertyDeviceUID, &size, &UI);
+       if (res == noErr) 
+               CFStringGetCString(UI,name,nsize,CFStringGetSystemEncoding());
+       CFRelease(UI);
+       return res;
+}
+
 vector<string>
 EngineControl::enumerate_coreaudio_devices ()
 {
@@ -485,9 +650,11 @@ EngineControl::enumerate_coreaudio_devices ()
        // Find out how many Core Audio devices are there, if any...
        // (code snippet gently "borrowed" from St?hane Letz jackdmp;)
        OSStatus err;
-       bool isWritable;
+       Boolean isWritable;
        size_t outSize = sizeof(isWritable);
 
+       backend_devs.clear ();
+
        err = AudioHardwareGetPropertyInfo(kAudioHardwarePropertyDevices,
                                           &outSize, &isWritable);
        if (err == noErr) {
@@ -500,8 +667,30 @@ EngineControl::enumerate_coreaudio_devices ()
                if (err == noErr) {
                        // Look for the CoreAudio device name...
                        char coreDeviceName[256];
-                       size_t nameSize = sizeof (coreDeviceName);
+                       size_t nameSize;
+
                        for (int i = 0; i < numCoreDevices; i++) {
+
+                               nameSize = sizeof (coreDeviceName);
+
+                               /* enforce duplex devices only */
+
+                               err = AudioDeviceGetPropertyInfo(coreDeviceIDs[i],
+                                                                0, true, kAudioDevicePropertyStreams,
+                                                                &outSize, &isWritable);
+
+                               if (err != noErr || outSize == 0) {
+                                       continue;
+                               }
+
+                               err = AudioDeviceGetPropertyInfo(coreDeviceIDs[i],
+                                                                0, false, kAudioDevicePropertyStreams,
+                                                                &outSize, &isWritable);
+
+                               if (err != noErr || outSize == 0) {
+                                       continue;
+                               }
+
                                err = AudioDeviceGetPropertyInfo(coreDeviceIDs[i],
                                                                 0, true, kAudioDevicePropertyDeviceName,
                                                                 &outSize, &isWritable);
@@ -510,31 +699,15 @@ EngineControl::enumerate_coreaudio_devices ()
                                                                     0, true, kAudioDevicePropertyDeviceName,
                                                                     &nameSize, (void *) coreDeviceName);
                                        if (err == noErr) {
-
                                                char drivername[128];
 
                                                // this returns the unique id for the device
                                                // that must be used on the commandline for jack
-
+                                               
                                                if (getDeviceUIDFromID(coreDeviceIDs[i], drivername, sizeof (drivername)) == noErr) {
-                                                       sName = drivername;
-                                               } else {
-                                                       sName = "Error";
-                                               }
-
-                                               devs.push_back (sName);
-#if 0                                          
-
-                                               coreaudioIdMap[sName] = coreDeviceIDs[i];
-                                               // TODO: hide this ugly ID from the user,
-                                               // only show human readable name
-                                               // humanreadable \t UID
-                                               sText = QString(coreDeviceName) + '\t' + sName;
-                                               pAction = menu.addAction(sText);
-                                               pAction->setCheckable(true);
-                                               pAction->setChecked(sCurName == sName);
-                                               ++iCards;
-#endif
+                                                       devs.push_back (coreDeviceName);
+                                                       backend_devs.push_back (drivername);
+                                               } 
                                        }
                                }
                        }
@@ -542,6 +715,28 @@ EngineControl::enumerate_coreaudio_devices ()
                delete [] coreDeviceIDs;
        }
 
+
+       if (devs.size() == 0) {
+               MessageDialog msg (_("\
+You do not have any audio devices capable of\n\
+simultaneous playback and recording.\n\n\
+Please use Applications -> Utilities -> Audio MIDI Setup\n\
+to create an \"aggregrate\" device, or install a suitable\n\
+audio interface.\n\n\
+Please send email to Apple and ask them why new Macs\n\
+have no duplex audio device.\n\n\
+Alternatively, if you really want just playback\n\
+or recording but not both, start JACK before running\n\
+Ardour and choose the relevant device then."
+                                          ), 
+                                  true, Gtk::MESSAGE_ERROR, Gtk::BUTTONS_OK);
+               msg.set_title (_("No suitable audio devices"));
+               msg.set_position (Gtk::WIN_POS_MOUSE);
+               msg.run ();
+               exit (1);
+       }
+
+
        return devs;
 }
 #else
@@ -549,18 +744,71 @@ vector<string>
 EngineControl::enumerate_alsa_devices ()
 {
        vector<string> devs;
-       devs.push_back ("hw:0");
-       devs.push_back ("hw:1");
-       devs.push_back ("plughw:0");
-       devs.push_back ("plughw:1");
+
+       snd_ctl_t *handle;
+       snd_ctl_card_info_t *info;
+       snd_pcm_info_t *pcminfo;
+       snd_ctl_card_info_alloca(&info);
+       snd_pcm_info_alloca(&pcminfo);
+       string devname;
+       int cardnum = -1;
+       int device = -1;
+
+       backend_devs.clear ();
+
+       while (snd_card_next (&cardnum) >= 0 && cardnum >= 0) {
+
+               devname = "hw:";
+               devname += to_string (cardnum, std::dec);
+
+               if (snd_ctl_open (&handle, devname.c_str(), 0) >= 0 && snd_ctl_card_info (handle, info) >= 0) {
+
+                       while (snd_ctl_pcm_next_device (handle, &device) >= 0 && device >= 0) {
+
+                               bool have_playback = false;
+                               bool have_capture = false;
+
+                               /* find duplex devices only */
+
+                               snd_pcm_info_set_device (pcminfo, device);
+                               snd_pcm_info_set_subdevice (pcminfo, 0);
+                               snd_pcm_info_set_stream (pcminfo, SND_PCM_STREAM_CAPTURE);
+
+                               if (snd_ctl_pcm_info (handle, pcminfo) >= 0) {
+                                       have_capture = true;
+                               }
+
+                               snd_pcm_info_set_device (pcminfo, device);
+                               snd_pcm_info_set_subdevice (pcminfo, 0);
+                               snd_pcm_info_set_stream (pcminfo, SND_PCM_STREAM_PLAYBACK);
+
+                               if (snd_ctl_pcm_info (handle, pcminfo) >= 0) {
+                                       have_playback = true;
+                               }
+
+                               if (have_capture && have_playback) {
+                                       devs.push_back (snd_pcm_info_get_name (pcminfo));
+                                       devname += ',';
+                                       devname += to_string (device, std::dec);
+                                       backend_devs.push_back (devname);
+                               }
+                       }
+
+                       snd_ctl_close(handle);
+               }
+       }
+
        return devs;
 }
+
 vector<string>
 EngineControl::enumerate_ffado_devices ()
 {
        vector<string> devs;
+       backend_devs.clear ();
        return devs;
 }
+
 vector<string>
 EngineControl::enumerate_oss_devices ()
 {
@@ -585,13 +833,35 @@ void
 EngineControl::driver_changed ()
 {
        string driver = driver_combo.get_active_text();
+       string::size_type maxlen = 0;
+       int maxindex = -1;
+       int n = 0;
+
+       enumerate_devices (driver);
+
        vector<string>& strings = devices[driver];
+
+       if (strings.empty() && driver != "FFADO" && driver != "Dummy") {
+               error << string_compose (_("No devices found for driver \"%1\""), driver) << endmsg;
+               return;
+       }
        
+       for (vector<string>::iterator i = strings.begin(); i != strings.end(); ++i, ++n) {
+               if ((*i).length() > maxlen) {
+                       maxlen = (*i).length();
+                       maxindex = n;
+               }
+       }
+
        set_popdown_strings (interface_combo, strings);
+       set_popdown_strings (input_device_combo, strings);
+       set_popdown_strings (output_device_combo, strings);
 
        if (!strings.empty()) {
                interface_combo.set_active_text (strings.front());
-       }
+               input_device_combo.set_active_text (strings.front());
+               output_device_combo.set_active_text (strings.front());
+       } 
        
        if (driver == "ALSA") {
                soft_mode_button.set_sensitive (true);
@@ -618,7 +888,11 @@ void
 EngineControl::redisplay_latency ()
 {
        uint32_t rate = get_rate();
+#ifdef __APPLE_
+       float periods = 2;
+#else
        float periods = periods_adjustment.get_value();
+#endif
        float period_size = atof (period_size_combo.get_active_text());
 
        char buf[32];
@@ -632,59 +906,375 @@ EngineControl::audio_mode_changed ()
 {
        Glib::ustring str = audio_mode_combo.get_active_text();
 
-       if (str == _("Duplex")) {
+       if (str == _("Playback/Recording on 1 Device")) {
                input_device_combo.set_sensitive (false);
                output_device_combo.set_sensitive (false);
-       } else {
+       } else if (str == _("Playback/Recording on 2 Devices")) {
                input_device_combo.set_sensitive (true);
                output_device_combo.set_sensitive (true);
+       } else if (str == _("Playback only")) {
+               output_device_combo.set_sensitive (true);
+       } else if (str == _("Recording only")) {
+               input_device_combo.set_sensitive (true);
        }
 }
 
+static bool jack_server_filter(const string& str, void *arg)
+{
+   return str == "jackd" || str == "jackdmp";
+}
+
 void
 EngineControl::find_jack_servers (vector<string>& strings)
 {
-#ifdef __APPLE
-       if (Profile->get_single_package()) {
+#ifdef __APPLE__
+       /* this magic lets us finds the path to the OSX bundle, and then
+          we infer JACK's location from there
+       */
+       
+       char execpath[MAXPATHLEN+1];
+       uint32_t pathsz = sizeof (execpath);
 
-               /* this magic lets us finds the path to the OSX bundle, and then
-                  we infer JACK's location from there
-               */
-               
-               CFURLRef pluginRef = CFBundleCopyBundleURL(CFBundleGetMainBundle());
-               CFStringRef macPath = CFURLCopyFileSystemPath(pluginRef, 
-                                                             kCFURLPOSIXPathStyle);
-               std::string path = CFStringGetCStringPtr(macPath, 
-                                                        CFStringGetSystemEncoding());
-               CFRelease(pluginRef);
-               CFRelease(macPath);
-               
-               path += '/jackd';
+       _NSGetExecutablePath (execpath, &pathsz);
+       
+       string path (Glib::path_get_dirname (execpath));
+       path += "/jackd";
+
+       if (Glib::file_test (path, FILE_TEST_EXISTS)) {
+               strings.push_back (path);
+       } 
+
+       if (getenv ("ARDOUR_WITH_JACK")) {
+               /* no other options - only use the JACK we supply */
+               if (strings.empty()) {
+                       fatal << _("JACK appears to be missing from the Ardour bundle") << endmsg;
+                       /*NOTREACHED*/
+               }
+               return;
+       }
+#else
+       string path;
+#endif
+       
+       PathScanner scanner;
+       vector<string *> *jack_servers;
+       std::map<string,int> un;
+       char *p;
+       bool need_minimal_path = false;
 
-               if (Glib::file_test (path, FILE_TEST_EXISTS)) {
-                       strings.push_back ();
+       p = getenv ("PATH");
+
+       if (p && *p) {
+               path = p;
+       } else {
+               need_minimal_path = true;
+       }
+
+#ifdef __APPLE__
+       // many mac users don't have PATH set up to include
+       // likely installed locations of JACK
+       need_minimal_path = true;
+#endif
+
+       if (need_minimal_path) {
+               if (path.empty()) {
+                       path = "/usr/bin:/bin:/usr/local/bin:/opt/local/bin";
                } else {
-                       warning << _("JACK appears to be missing from the Ardour bundle") << endmsg;
+                       path += ":/usr/local/bin:/opt/local/bin";
                }
+       }
+
+#ifdef __APPLE__
+       // push it back into the environment so that auto-started JACK can find it.
+       // XXX why can't we just expect OS X users to have PATH set correctly? we can't ...
+       setenv ("PATH", path.c_str(), 1);
 #endif
 
-       if (Glib::file_test ("/usr/bin/jackd", FILE_TEST_EXISTS)) {
-               strings.push_back ("/usr/bin/jackd");
+       jack_servers = scanner (path, jack_server_filter, 0, false, true);
+       
+       vector<string *>::iterator iter;
+       
+       for (iter = jack_servers->begin(); iter != jack_servers->end(); iter++) {
+               string p = **iter;
+               
+               if (un[p]++ == 0) {
+                       strings.push_back(p);
+               }
        }
-       if (Glib::file_test ("/usr/local/bin/jackd", FILE_TEST_EXISTS)) {
-               strings.push_back ("/usr/local/bin/jackd");
+}
+
+string
+EngineControl::get_device_name (const string& driver, const string& human_readable)
+{
+       vector<string>::iterator n;
+       vector<string>::iterator i;
+
+       if (human_readable.empty()) {
+               /* this can happen if the user's .ardourrc file has a device name from
+                  another computer system in it
+               */
+               MessageDialog msg (_("You need to choose an audio device first."));
+               msg.run ();
+               return string();
        }
-       if (Glib::file_test ("/opt/bin/jackd", FILE_TEST_EXISTS)) {
-               strings.push_back ("/opt/bin/jackd");
+
+       if (backend_devs.empty()) {
+               return human_readable;
        }
-       if (Glib::file_test ("/usr/bin/jackdmp", FILE_TEST_EXISTS)) {
-               strings.push_back ("/usr/bin/jackd");
+       
+       for (i = devices[driver].begin(), n = backend_devs.begin(); i != devices[driver].end(); ++i, ++n) {
+               if (human_readable == (*i)) {
+                       return (*n);
+               }
        }
-       if (Glib::file_test ("/usr/local/bin/jackdmp", FILE_TEST_EXISTS)) {
-               strings.push_back ("/usr/local/bin/jackd");
+       
+       if (i == devices[driver].end()) {
+               warning << string_compose (_("Audio device \"%1\" not known on this computer."), human_readable) << endmsg;
        }
-       if (Glib::file_test ("/opt/bin/jackdmp", FILE_TEST_EXISTS)) {
-               strings.push_back ("/opt/bin/jackd");
+
+       return string();
+}
+
+XMLNode&
+EngineControl::get_state ()
+{
+       XMLNode* root = new XMLNode ("AudioSetup");
+       XMLNode* child;
+       Glib::ustring path;
+
+       child = new XMLNode ("periods");
+       child->add_property ("val", to_string (periods_adjustment.get_value(), std::dec));
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("priority");
+       child->add_property ("val", to_string (priority_adjustment.get_value(), std::dec));
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("ports");
+       child->add_property ("val", to_string (ports_adjustment.get_value(), std::dec));
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("inchannels");
+       child->add_property ("val", to_string (input_channels.get_value(), std::dec));
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("outchannels");
+       child->add_property ("val", to_string (output_channels.get_value(), std::dec));
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("inlatency");
+       child->add_property ("val", to_string (input_latency.get_value(), std::dec));
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("outlatency");
+       child->add_property ("val", to_string (output_latency.get_value(), std::dec));
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("realtime");
+       child->add_property ("val", to_string (realtime_button.get_active(), std::dec));
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("nomemorylock");
+       child->add_property ("val", to_string (no_memory_lock_button.get_active(), std::dec));
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("unlockmemory");
+       child->add_property ("val", to_string (unlock_memory_button.get_active(), std::dec));
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("softmode");
+       child->add_property ("val", to_string (soft_mode_button.get_active(), std::dec));
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("force16bit");
+       child->add_property ("val", to_string (force16bit_button.get_active(), std::dec));
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("hwmonitor");
+       child->add_property ("val", to_string (hw_monitor_button.get_active(), std::dec));
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("hwmeter");
+       child->add_property ("val", to_string (hw_meter_button.get_active(), std::dec));
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("verbose");
+       child->add_property ("val", to_string (verbose_output_button.get_active(), std::dec));
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("samplerate");
+       child->add_property ("val", sample_rate_combo.get_active_text());
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("periodsize");
+       child->add_property ("val", period_size_combo.get_active_text());
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("serverpath");
+       child->add_property ("val", serverpath_combo.get_active_text());
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("driver");
+       child->add_property ("val", driver_combo.get_active_text());
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("interface");
+       child->add_property ("val", interface_combo.get_active_text());
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("timeout");
+       child->add_property ("val", timeout_combo.get_active_text());
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("dither");
+       child->add_property ("val", dither_mode_combo.get_active_text());
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("audiomode");
+       child->add_property ("val", audio_mode_combo.get_active_text());
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("inputdevice");
+       child->add_property ("val", input_device_combo.get_active_text());
+       root->add_child_nocopy (*child);
+
+       child = new XMLNode ("outputdevice");
+       child->add_property ("val", output_device_combo.get_active_text());
+       root->add_child_nocopy (*child);
+       
+       return *root;
+}
+
+void
+EngineControl::set_state (const XMLNode& root)
+{
+       XMLNodeList          clist;
+       XMLNodeConstIterator citer;
+       XMLNode* child;
+       XMLProperty* prop;
+       bool using_dummy = false;
+       
+       int val;
+       string strval;
+       
+       if ( (child = root.child ("driver"))){
+               prop = child->property("val");
+               if (prop && (prop->value() == "Dummy") ) {
+                       using_dummy = true;
+               }
        }
+       
+       clist = root.children();
 
+       for (citer = clist.begin(); citer != clist.end(); ++citer) {
+               if ( prop && (prop->value() == "FFADO" ))
+                               continue;
+               child = *citer;
+
+               prop = child->property ("val");
+
+               if (!prop || prop->value().empty()) {
+
+                       if ( using_dummy && ( child->name() == "interface" || child->name() == "inputdevice" || child->name() == "outputdevice" ))
+                               continue;
+                       error << string_compose (_("AudioSetup value for %1 is missing data"), child->name()) << endmsg;
+                       continue;
+               }
+               
+               strval = prop->value();
+
+               /* adjustments/spinners */
+
+               if (child->name() == "periods") {
+                       val = atoi (strval);
+                       periods_adjustment.set_value(val);
+               } else if (child->name() == "priority") {
+                       val = atoi (strval);
+                       priority_adjustment.set_value(val);
+               } else if (child->name() == "ports") {
+                       val = atoi (strval);
+                       ports_adjustment.set_value(val);
+               } else if (child->name() == "inchannels") {
+                       val = atoi (strval);
+                       input_channels.set_value(val);
+               } else if (child->name() == "outchannels") {
+                       val = atoi (strval);
+                       output_channels.set_value(val);
+               } else if (child->name() == "inlatency") {
+                       val = atoi (strval);
+                       input_latency.set_value(val);
+               } else if (child->name() == "outlatency") {
+                       val = atoi (strval);
+                       output_latency.set_value(val);
+               }
+
+               /* buttons */
+
+               else if (child->name() == "realtime") {
+                       val = atoi (strval);
+                       realtime_button.set_active(val);
+               } else if (child->name() == "nomemorylock") {
+                       val = atoi (strval);
+                       no_memory_lock_button.set_active(val);
+               } else if (child->name() == "unlockmemory") {
+                       val = atoi (strval);
+                       unlock_memory_button.set_active(val);
+               } else if (child->name() == "softmode") {
+                       val = atoi (strval);
+                       soft_mode_button.set_active(val);
+               } else if (child->name() == "force16bit") {
+                       val = atoi (strval);
+                       force16bit_button.set_active(val);
+               } else if (child->name() == "hwmonitor") {
+                       val = atoi (strval);
+                       hw_monitor_button.set_active(val);
+               } else if (child->name() == "hwmeter") {
+                       val = atoi (strval);
+                       hw_meter_button.set_active(val);
+               } else if (child->name() == "verbose") {
+                       val = atoi (strval);
+                       verbose_output_button.set_active(val);
+               }
+
+               /* combos */
+
+               else if (child->name() == "samplerate") {
+                       sample_rate_combo.set_active_text(strval);
+               } else if (child->name() == "periodsize") {
+                       period_size_combo.set_active_text(strval);
+               } else if (child->name() == "serverpath") {
+                       /* do not allow us to use a server path that doesn't
+                          exist on this system. this handles cases where
+                          the user has an RC file listing a serverpath
+                          from some other machine.
+                       */
+                       vector<string>::iterator x;
+                       for (x = server_strings.begin(); x != server_strings.end(); ++x) {
+                               if (*x == strval) {
+                                       break;
+                               }
+                       }
+                       if (x != server_strings.end()) {
+                               serverpath_combo.set_active_text (strval);
+                       } else {
+                               warning << string_compose (_("configuration files contain a JACK server path that doesn't exist (%1)"),
+                                                            strval)
+                                       << endmsg;
+                       }
+               } else if (child->name() == "driver") {
+                       driver_combo.set_active_text(strval);
+               } else if (child->name() == "interface") {
+                       interface_combo.set_active_text(strval);
+               } else if (child->name() == "timeout") {
+                       timeout_combo.set_active_text(strval);
+               } else if (child->name() == "dither") {
+                       dither_mode_combo.set_active_text(strval);
+               } else if (child->name() == "audiomode") {
+                       audio_mode_combo.set_active_text(strval);
+               } else if (child->name() == "inputdevice") {
+                       input_device_combo.set_active_text(strval);
+               } else if (child->name() == "outputdevice") {
+                       output_device_combo.set_active_text(strval);
+               }
+       }
 }