changes to help strp silence
[ardour.git] / gtk2_ardour / rc_option_editor.cc
index 390ebe92de0a4faa7badd4d7538ffa60789d899b..f9432bf88b8d45d2001221aa88ab9cfa090dc8ae 100644 (file)
@@ -2,12 +2,17 @@
 #include <gtkmm/stock.h>
 #include <gtkmm/scale.h>
 #include <gtkmm2ext/utils.h>
+#include <gtkmm2ext/slider_controller.h>
 #include "pbd/fpu.h"
 #include "midi++/manager.h"
 #include "midi++/factory.h"
+#include "ardour/dB.h"
 #include "ardour/rc_configuration.h"
 #include "ardour/control_protocol_manager.h"
 #include "control_protocol/control_protocol.h"
+
+#include "gui_thread.h"
+#include "midi_tracer.h"
 #include "rc_option_editor.h"
 #include "utils.h"
 #include "midi_port_dialog.h"
@@ -16,7 +21,6 @@
 #include "i18n.h"
 
 using namespace std;
-using namespace sigc;
 using namespace Gtk;
 using namespace Gtkmm2ext;
 using namespace PBD;
@@ -25,9 +29,10 @@ using namespace ARDOUR;
 class MIDIPorts : public OptionEditorBox
 {
 public:
-       MIDIPorts (RCConfiguration* c)
+       MIDIPorts (RCConfiguration* c, list<ComboOption<string>* > const & o)
                : _rc_config (c),
-                 _add_port_button (Stock::ADD)
+                 _add_port_button (Stock::ADD),
+                 _port_combos (o)
        {
                _store = ListStore::create (_model);
                _view.set_model (_store);
@@ -49,91 +54,23 @@ public:
 
                _box->pack_start (*h);
 
-               Table* t = manage (new Table (2, 4));
-               t->set_spacings (12);
-
-               int n = 0;
-               Label* l = manage (new Label (_("MTC:")));
-               l->set_alignment (1, 0.5);
-               t->attach (*l, 0, 1, n, n + 1, EXPAND | FILL, FILL);
-               t->attach (_mtc_combo, 1, 2, n, n + 1, EXPAND | FILL, EXPAND | FILL);
-               ++n;
-               
-               l = manage (new Label (_("MIDI clock:")));
-               l->set_alignment (1, 0.5);
-               t->attach (*l, 0, 1, n, n + 1, FILL, FILL);
-               t->attach (_midi_clock_combo, 1, 2, n, n + 1, FILL, FILL);
-               ++n;
-
-               l = manage (new Label (_("MMC:")));
-               l->set_alignment (1, 0.5);
-               t->attach (*l, 0, 1, n, n + 1, FILL, FILL);
-               t->attach (_mmc_combo, 1, 2, n, n + 1, FILL, FILL);
-               ++n;
-
-               l = manage (new Label (_("MIDI parameter control:")));
-               l->set_alignment (1, 0.5);
-               t->attach (*l, 0, 1, n, n + 1, FILL, FILL);
-               t->attach (_mpc_combo, 1, 2, n, n + 1, FILL, FILL);
-               ++n;
-
-               _box->pack_start (*t, true, true);
-
                ports_changed ();
 
-               _store->signal_row_changed().connect (mem_fun (*this, &MIDIPorts::model_changed));
-
-               _add_port_button.signal_clicked().connect (mem_fun (*this, &MIDIPorts::add_port_clicked));
-               _mtc_combo.signal_changed().connect (mem_fun (*this, &MIDIPorts::mtc_combo_changed));
-               _mmc_combo.signal_changed().connect (mem_fun (*this, &MIDIPorts::mmc_combo_changed));
-               _mpc_combo.signal_changed().connect (mem_fun (*this, &MIDIPorts::mpc_combo_changed));
-               _midi_clock_combo.signal_changed().connect (mem_fun (*this, &MIDIPorts::midi_clock_combo_changed));
-       }
-
-       void parameter_changed (string const & p)
-       {
-               if (p == "mtc-port-name") {
-                       _mtc_combo.set_active_text (_rc_config->get_mtc_port_name());
-               } else if (p == "mmc-port-name") {
-                       _mmc_combo.set_active_text (_rc_config->get_mmc_port_name());
-               } else if (p == "midi-port-name") {
-                       _mpc_combo.set_active_text (_rc_config->get_midi_port_name());
-               } else if (p == "midi-clock-port-name") {
-                       _midi_clock_combo.set_active_text (_rc_config->get_midi_clock_port_name());
-               } 
-       }
-
-       void set_state_from_config ()
-       {
-               parameter_changed ("mtc-port-name");
-               parameter_changed ("mmc-port-name");
-               parameter_changed ("midi-port-name");
-               parameter_changed ("midi-clock-port-name");
-       }
-
-       void mtc_combo_changed ()
-       {
-               _rc_config->set_mtc_port_name (_mtc_combo.get_active_text());
-       }
+               _store->signal_row_changed().connect (sigc::mem_fun (*this, &MIDIPorts::model_changed));
 
-       void mmc_combo_changed ()
-       {
-               _rc_config->set_mmc_port_name (_mmc_combo.get_active_text());
+               _add_port_button.signal_clicked().connect (sigc::mem_fun (*this, &MIDIPorts::add_port_clicked));
        }
 
-       void mpc_combo_changed ()
-       {
-               _rc_config->set_midi_port_name (_mpc_combo.get_active_text());
-       }
+       void parameter_changed (string const &) {}
+       void set_state_from_config () {}
 
-       void midi_clock_combo_changed ()
-       {
-               _rc_config->set_midi_clock_port_name (_midi_clock_combo.get_active_text());
-       }
-       
 private:
 
-       void model_changed (TreeModel::Path const & p, TreeModel::iterator const & i)
+        typedef std::map<MIDI::Port*,MidiTracer*> PortTraceMap;
+        PortTraceMap port_input_trace_map;
+        PortTraceMap port_output_trace_map;
+
+       void model_changed (TreeModel::Path const &, TreeModel::iterator const & i)
        {
                TreeModel::Row r = *i;
 
@@ -143,67 +80,82 @@ private:
                }
 
                if (port->input()) {
-                       
+
                        if (r[_model.online] == port->input()->offline()) {
                                port->input()->set_offline (!r[_model.online]);
                        }
 
                        if (r[_model.trace_input] != port->input()->tracing()) {
-                               port->input()->trace (r[_model.trace_input], &cerr, string (port->name()) + _(" input: "));
+                               PortTraceMap::iterator x = port_input_trace_map.find (port);
+                               MidiTracer* mt;
+
+                               if (x == port_input_trace_map.end()) {
+                                        mt = new MidiTracer (port->name() + string (" [input]"), *port->input());
+                                        port_input_trace_map.insert (pair<MIDI::Port*,MidiTracer*> (port, mt));
+                               } else {
+                                       mt = x->second;
+                               }
+                               mt->present ();
                        }
                }
 
                if (port->output()) {
 
                        if (r[_model.trace_output] != port->output()->tracing()) {
-                               port->output()->trace (r[_model.trace_output], &cerr, string (port->name()) + _(" output: "));
+                               PortTraceMap::iterator x = port_output_trace_map.find (port);
+                               MidiTracer* mt;
+
+                               if (x == port_output_trace_map.end()) {
+                                       mt = new MidiTracer (port->name() + string (" [output]"), *port->output());
+                                       port_output_trace_map.insert (pair<MIDI::Port*,MidiTracer*> (port, mt));
+                               } else {
+                                       mt = x->second;
+                               }
+                               mt->present ();
                        }
-                       
-               }
 
-               
-                               
+               }
        }
 
-       void setup_ports_combo (ComboBoxText& c)
+       void setup_ports_combo (ComboOption<string>* c)
        {
-               c.clear_items ();
-               MIDI::Manager::PortMap const & ports = MIDI::Manager::instance()->get_midi_ports ();
-               for (MIDI::Manager::PortMap::const_iterator i = ports.begin(); i != ports.end(); ++i) {
-                       c.append_text (i->first);
+               c->clear ();
+               MIDI::Manager::PortList const & ports = MIDI::Manager::instance()->get_midi_ports ();
+               for (MIDI::Manager::PortList::const_iterator i = ports.begin(); i != ports.end(); ++i) {
+                       c->add ((*i)->name(), (*i)->name());
                }
-       }
+       }       
 
        void ports_changed ()
        {
                /* XXX: why is this coming from here? */
-               MIDI::Manager::PortMap const & ports = MIDI::Manager::instance()->get_midi_ports ();
+               MIDI::Manager::PortList const & ports = MIDI::Manager::instance()->get_midi_ports ();
 
                _store->clear ();
+               port_connections.drop_connections ();
 
-               for (MIDI::Manager::PortMap::const_iterator i = ports.begin(); i != ports.end(); ++i) {
+               for (MIDI::Manager::PortList::const_iterator i = ports.begin(); i != ports.end(); ++i) {
 
                        TreeModel::Row r = *_store->append ();
 
-                       r[_model.name] = i->first;
+                       r[_model.name] = (*i)->name();
 
-                       if (i->second->input()) {
-                               r[_model.online] = !i->second->input()->offline();
-                               i->second->input()->OfflineStatusChanged.connect (bind (mem_fun (*this, &MIDIPorts::port_offline_changed), i->second));
-                               r[_model.trace_input] = i->second->input()->tracing();
+                       if ((*i)->input()) {
+                               r[_model.online] = !(*i)->input()->offline();
+                               (*i)->input()->OfflineStatusChanged.connect (port_connections, boost::bind (&MIDIPorts::port_offline_changed, this, (*i)), gui_context());
+                               r[_model.trace_input] = (*i)->input()->tracing();
                        }
 
-                       if (i->second->output()) {
-                               r[_model.trace_output] = i->second->output()->tracing();
+                       if ((*i)->output()) {
+                               r[_model.trace_output] = (*i)->output()->tracing();
                        }
 
-                       r[_model.port] = i->second;
+                       r[_model.port] = (*i);
                }
 
-               setup_ports_combo (_mtc_combo);
-               setup_ports_combo (_midi_clock_combo);
-               setup_ports_combo (_mmc_combo);
-               setup_ports_combo (_mpc_combo);
+               for (list<ComboOption<string>* >::iterator i = _port_combos.begin(); i != _port_combos.end(); ++i) {
+                       setup_ports_combo (*i);
+               }
        }
 
        void port_offline_changed (MIDI::Port* p)
@@ -226,7 +178,7 @@ private:
                dialog.set_position (WIN_POS_MOUSE);
 
                dialog.show ();
-               
+
                int const r = dialog.run ();
 
                switch (r) {
@@ -249,17 +201,18 @@ private:
                }
 
                XMLNode node (X_("MIDI-port"));
-               
+
                node.add_property ("tag", dialog.port_name.get_text());
                node.add_property ("device", X_("ardour")); // XXX this can't be right for all types
                node.add_property ("type", MIDI::PortFactory::default_port_type());
                node.add_property ("mode", smod);
-               
+
                if (MIDI::Manager::instance()->add_port (node) != 0) {
+                       cerr << " there are now " << MIDI::Manager::instance()->nports() << endl;
                        ports_changed ();
                }
        }
-       
+
        class MIDIModelColumns : public TreeModelColumnRecord
        {
        public:
@@ -288,6 +241,8 @@ private:
        ComboBoxText _midi_clock_combo;
        ComboBoxText _mmc_combo;
        ComboBoxText _mpc_combo;
+       list<ComboOption<string>* > _port_combos;
+        PBD::ScopedConnectionList port_connections;
 };
 
 
@@ -302,24 +257,24 @@ public:
                t->set_spacings (4);
 
                Label* l = manage (new Label (_("Click audio file:")));
-               l->set_alignment (1, 0.5);
+               l->set_alignment (0, 0.5);
                t->attach (*l, 0, 1, 0, 1, FILL);
                t->attach (_click_path_entry, 1, 2, 0, 1, FILL);
                Button* b = manage (new Button (_("Browse...")));
-               b->signal_clicked().connect (mem_fun (*this, &ClickOptions::click_browse_clicked));
+               b->signal_clicked().connect (sigc::mem_fun (*this, &ClickOptions::click_browse_clicked));
                t->attach (*b, 2, 3, 0, 1, FILL);
 
                l = manage (new Label (_("Click emphasis audio file:")));
-               l->set_alignment (1, 0.5);
+               l->set_alignment (0, 0.5);
                t->attach (*l, 0, 1, 1, 2, FILL);
                t->attach (_click_emphasis_path_entry, 1, 2, 1, 2, FILL);
                b = manage (new Button (_("Browse...")));
-               b->signal_clicked().connect (mem_fun (*this, &ClickOptions::click_emphasis_browse_clicked));
+               b->signal_clicked().connect (sigc::mem_fun (*this, &ClickOptions::click_emphasis_browse_clicked));
                t->attach (*b, 2, 3, 1, 2, FILL);
 
                _box->pack_start (*t, false, false);
        }
-       
+
        void parameter_changed (string const & p)
        {
                if (p == "click-sound") {
@@ -335,7 +290,7 @@ public:
                parameter_changed ("click-emphasis-sound");
        }
 
-private:       
+private:
 
        void click_browse_clicked ()
        {
@@ -343,7 +298,7 @@ private:
 
                sfdb.show_all ();
                sfdb.present ();
-               
+
                if (sfdb.run () == RESPONSE_OK) {
                        click_chosen (sfdb.get_filename());
                }
@@ -361,7 +316,7 @@ private:
 
                sfdb.show_all ();
                sfdb.present ();
-               
+
                if (sfdb.run () == RESPONSE_OK) {
                        click_emphasis_chosen (sfdb.get_filename());
                }
@@ -408,10 +363,10 @@ public:
 
                _box->pack_start (*t);
 
-               _limit_undo_button.signal_toggled().connect (mem_fun (*this, &UndoOptions::limit_undo_toggled));
-               _limit_undo_spin.signal_value_changed().connect (mem_fun (*this, &UndoOptions::limit_undo_changed));
-               _save_undo_button.signal_toggled().connect (mem_fun (*this, &UndoOptions::save_undo_toggled));
-               _save_undo_spin.signal_value_changed().connect (mem_fun (*this, &UndoOptions::save_undo_changed));
+               _limit_undo_button.signal_toggled().connect (sigc::mem_fun (*this, &UndoOptions::limit_undo_toggled));
+               _limit_undo_spin.signal_value_changed().connect (sigc::mem_fun (*this, &UndoOptions::limit_undo_changed));
+               _save_undo_button.signal_toggled().connect (sigc::mem_fun (*this, &UndoOptions::save_undo_toggled));
+               _save_undo_spin.signal_value_changed().connect (sigc::mem_fun (*this, &UndoOptions::save_undo_changed));
        }
 
        void parameter_changed (string const & p)
@@ -461,7 +416,7 @@ public:
        {
                _rc_config->set_saved_history_depth (_save_undo_spin.get_value_as_int ());
        }
-       
+
 private:
        RCConfiguration* _rc_config;
        CheckButton _limit_undo_button;
@@ -477,12 +432,13 @@ static const struct {
     guint modifier;
 } modifiers[] = {
 
+       { "Unmodified", 0 },
+
 #ifdef GTKOSX
 
        /* Command = Meta
           Option/Alt = Mod1
        */
-
        { "Shift", GDK_SHIFT_MASK },
        { "Command", GDK_META_MASK },
        { "Control", GDK_CONTROL_MASK },
@@ -513,11 +469,11 @@ class KeyboardOptions : public OptionEditorBox
 {
 public:
        KeyboardOptions () :
-                 _delete_button_adjustment (3, 1, 5),
+                 _delete_button_adjustment (3, 1, 12),
                  _delete_button_spin (_delete_button_adjustment),
                  _edit_button_adjustment (3, 1, 5),
                  _edit_button_spin (_edit_button_adjustment)
-               
+
        {
                /* internationalize and prepare for use with combos */
 
@@ -527,7 +483,7 @@ public:
                }
 
                set_popdown_strings (_edit_modifier_combo, dumb);
-               _edit_modifier_combo.signal_changed().connect (mem_fun(*this, &KeyboardOptions::edit_modifier_chosen));
+               _edit_modifier_combo.signal_changed().connect (sigc::mem_fun(*this, &KeyboardOptions::edit_modifier_chosen));
 
                for (int x = 0; modifiers[x].name; ++x) {
                        if (modifiers[x].modifier == Keyboard::edit_modifier ()) {
@@ -541,7 +497,7 @@ public:
 
                Label* l = manage (new Label (_("Edit using:")));
                l->set_name ("OptionsLabel");
-               l->set_alignment (1.0, 0.5);
+               l->set_alignment (0, 0.5);
 
                t->attach (*l, 0, 1, 0, 1, FILL | EXPAND, FILL);
                t->attach (_edit_modifier_combo, 1, 2, 0, 1, FILL | EXPAND, FILL);
@@ -554,10 +510,10 @@ public:
 
                _edit_button_spin.set_name ("OptionsEntry");
                _edit_button_adjustment.set_value (Keyboard::edit_button());
-               _edit_button_adjustment.signal_value_changed().connect (mem_fun(*this, &KeyboardOptions::edit_button_changed));
+               _edit_button_adjustment.signal_value_changed().connect (sigc::mem_fun(*this, &KeyboardOptions::edit_button_changed));
 
                set_popdown_strings (_delete_modifier_combo, dumb);
-               _delete_modifier_combo.signal_changed().connect (mem_fun(*this, &KeyboardOptions::delete_modifier_chosen));
+               _delete_modifier_combo.signal_changed().connect (sigc::mem_fun(*this, &KeyboardOptions::delete_modifier_chosen));
 
                for (int x = 0; modifiers[x].name; ++x) {
                        if (modifiers[x].modifier == Keyboard::delete_modifier ()) {
@@ -568,7 +524,7 @@ public:
 
                l = manage (new Label (_("Delete using:")));
                l->set_name ("OptionsLabel");
-               l->set_alignment (1.0, 0.5);
+               l->set_alignment (0, 0.5);
 
                t->attach (*l, 0, 1, 1, 2, FILL | EXPAND, FILL);
                t->attach (_delete_modifier_combo, 1, 2, 1, 2, FILL | EXPAND, FILL);
@@ -581,10 +537,10 @@ public:
 
                _delete_button_spin.set_name ("OptionsEntry");
                _delete_button_adjustment.set_value (Keyboard::delete_button());
-               _delete_button_adjustment.signal_value_changed().connect (mem_fun(*this, &KeyboardOptions::delete_button_changed));
+               _delete_button_adjustment.signal_value_changed().connect (sigc::mem_fun(*this, &KeyboardOptions::delete_button_changed));
 
                set_popdown_strings (_snap_modifier_combo, dumb);
-               _snap_modifier_combo.signal_changed().connect (mem_fun(*this, &KeyboardOptions::snap_modifier_chosen));
+               _snap_modifier_combo.signal_changed().connect (sigc::mem_fun(*this, &KeyboardOptions::snap_modifier_chosen));
 
                for (int x = 0; modifiers[x].name; ++x) {
                        if (modifiers[x].modifier == (guint) Keyboard::snap_modifier ()) {
@@ -593,9 +549,9 @@ public:
                        }
                }
 
-               l = manage (new Label (_("Ignore snap using:")));
+               l = manage (new Label (_("Toggle snap using:")));
                l->set_name ("OptionsLabel");
-               l->set_alignment (1.0, 0.5);
+               l->set_alignment (0, 0.5);
 
                t->attach (*l, 0, 1, 2, 3, FILL | EXPAND, FILL);
                t->attach (_snap_modifier_combo, 1, 2, 2, 3, FILL | EXPAND, FILL);
@@ -608,11 +564,11 @@ public:
 
                set_popdown_strings (_keyboard_layout_selector, strs);
                _keyboard_layout_selector.set_active_text (Keyboard::current_binding_name());
-               _keyboard_layout_selector.signal_changed().connect (mem_fun (*this, &KeyboardOptions::bindings_changed));
+               _keyboard_layout_selector.signal_changed().connect (sigc::mem_fun (*this, &KeyboardOptions::bindings_changed));
 
                l = manage (new Label (_("Keyboard layout:")));
                l->set_name ("OptionsLabel");
-               l->set_alignment (1.0, 0.5);
+               l->set_alignment (0, 0.5);
 
                t->attach (*l, 0, 1, 3, 4, FILL | EXPAND, FILL);
                t->attach (_keyboard_layout_selector, 1, 2, 3, 4, FILL | EXPAND, FILL);
@@ -620,7 +576,7 @@ public:
                _box->pack_start (*t, false, false);
        }
 
-       void parameter_changed (string const & p)
+       void parameter_changed (string const &)
        {
                /* XXX: these aren't really config options... */
        }
@@ -637,7 +593,7 @@ private:
                string const txt = _keyboard_layout_selector.get_active_text();
 
                /* XXX: config...?  for all this keyboard stuff */
-               
+
                for (map<string,string>::iterator i = Keyboard::binding_files.begin(); i != Keyboard::binding_files.end(); ++i) {
                        if (txt == i->first) {
                                if (Keyboard::load_keybindings (i->second)) {
@@ -650,7 +606,7 @@ private:
        void edit_modifier_chosen ()
        {
                string const txt = _edit_modifier_combo.get_active_text();
-               
+
                for (int i = 0; modifiers[i].name; ++i) {
                        if (txt == _(modifiers[i].name)) {
                                Keyboard::set_edit_modifier (modifiers[i].modifier);
@@ -662,7 +618,7 @@ private:
        void delete_modifier_chosen ()
        {
                string const txt = _delete_modifier_combo.get_active_text();
-               
+
                for (int i = 0; modifiers[i].name; ++i) {
                        if (txt == _(modifiers[i].name)) {
                                Keyboard::set_delete_modifier (modifiers[i].modifier);
@@ -670,11 +626,11 @@ private:
                        }
                }
        }
-       
+
        void snap_modifier_chosen ()
        {
                string const txt = _snap_modifier_combo.get_active_text();
-               
+
                for (int i = 0; modifiers[i].name; ++i) {
                        if (txt == _(modifiers[i].name)) {
                                Keyboard::set_snap_modifier (modifiers[i].modifier);
@@ -692,7 +648,7 @@ private:
        {
                Keyboard::set_edit_button (_edit_button_spin.get_value_as_int());
        }
-       
+
        ComboBoxText _keyboard_layout_selector;
        ComboBoxText _edit_modifier_combo;
        ComboBoxText _delete_modifier_combo;
@@ -711,7 +667,7 @@ public:
                _dpi_adjustment (50, 50, 250, 1, 10),
                _dpi_slider (_dpi_adjustment)
        {
-               _dpi_adjustment.set_value (_rc_config->get_font_scale ());
+               _dpi_adjustment.set_value (_rc_config->get_font_scale () / 1024);
 
                Label* l = manage (new Label (_("Font scaling:")));
                l->set_name ("OptionsLabel");
@@ -724,13 +680,13 @@ public:
 
                _box->pack_start (*h, false, false);
 
-               _dpi_adjustment.signal_value_changed().connect (mem_fun (*this, &FontScalingOptions::dpi_changed));
+               _dpi_adjustment.signal_value_changed().connect (sigc::mem_fun (*this, &FontScalingOptions::dpi_changed));
        }
 
        void parameter_changed (string const & p)
        {
                if (p == "font-scale") {
-                       _dpi_adjustment.set_value (_rc_config->get_font_scale() / 1024.);
+                       _dpi_adjustment.set_value (_rc_config->get_font_scale() / 1024);
                }
        }
 
@@ -738,9 +694,9 @@ public:
        {
                parameter_changed ("font-scale");
        }
-       
+
 private:
-       
+
        void dpi_changed ()
        {
                _rc_config->set_font_scale ((long) floor (_dpi_adjustment.get_value() * 1024));
@@ -753,11 +709,87 @@ private:
        HScale _dpi_slider;
 };
 
+class SoloMuteOptions : public OptionEditorBox
+{
+public:
+       SoloMuteOptions (RCConfiguration* c) :
+               _rc_config (c),
+               // 0.781787 is the value needed for gain to be set to 0.
+               _db_adjustment (0.781787, 0.0, 1.0, 0.01, 0.1)
+
+       {
+               if ((pix = ::get_icon ("fader_belt_h")) == 0) {
+                       throw failed_constructor();
+               }
+
+               _db_slider = manage (new HSliderController (pix,
+                                                           &_db_adjustment,
+                                                           false,
+                                                           115));
+
+
+               parameter_changed ("solo-mute-gain");
+
+               Label* l = manage (new Label (_("Solo mute cut (dB):")));
+               l->set_name ("OptionsLabel");
+
+               HBox* h = manage (new HBox);
+               h->set_spacing (4);
+               h->pack_start (*l, false, false);
+               h->pack_start (*_db_slider, false, false);
+               h->pack_start (_db_display, false, false);
+
+               set_size_request_to_display_given_text (_db_display, "-99.0", 12, 12);
+
+               _box->pack_start (*h, false, false);
+
+               _db_adjustment.signal_value_changed().connect (sigc::mem_fun (*this, &SoloMuteOptions::db_changed));
+       }
+
+       void parameter_changed (string const & p)
+       {
+               if (p == "solo-mute-gain") {
+                       gain_t val = _rc_config->get_solo_mute_gain();
+
+                       _db_adjustment.set_value (gain_to_slider_position (val));
+
+                       char buf[16];
+
+                       if (val == 0.0) {
+                               snprintf (buf, sizeof (buf), "-inf");
+                       } else {
+                               snprintf (buf, sizeof (buf), "%.2f", accurate_coefficient_to_dB (val));
+                       }
+
+                       _db_display.set_text (buf);
+               }
+       }
+
+       void set_state_from_config ()
+       {
+               parameter_changed ("solo-mute-gain");
+       }
+
+private:
+
+       void db_changed ()
+       {
+               _rc_config->set_solo_mute_gain (slider_position_to_gain (_db_adjustment.get_value()));
+       }
+
+       RCConfiguration* _rc_config;
+       Adjustment _db_adjustment;
+        Gtkmm2ext::HSliderController* _db_slider;
+        Glib::RefPtr<Gdk::Pixbuf> pix;
+        Entry _db_display;
+};
+
 
 class ControlSurfacesOptions : public OptionEditorBox
 {
 public:
-       ControlSurfacesOptions ()
+       ControlSurfacesOptions (ArdourDialog& parent)
+               : _parent (parent)
        {
                _store = ListStore::create (_model);
                _view.set_model (_store);
@@ -769,7 +801,14 @@ public:
 
                _box->pack_start (_view, false, false);
 
-               _store->signal_row_changed().connect (mem_fun (*this, &ControlSurfacesOptions::model_changed));
+               Label* label = manage (new Label);
+               label->set_markup (string_compose (X_("<i>%1</i>"), _("Double-click on a name to edit settings for an enabled protocol")));
+
+               _box->pack_start (*label, false, false);
+               label->show ();
+               
+               _store->signal_row_changed().connect (sigc::mem_fun (*this, &ControlSurfacesOptions::model_changed));
+               _view.signal_button_press_event().connect_notify (sigc::mem_fun(*this, &ControlSurfacesOptions::edit_clicked));
        }
 
        void parameter_changed (std::string const &)
@@ -780,7 +819,7 @@ public:
        void set_state_from_config ()
        {
                _store->clear ();
-               
+
                ControlProtocolManager& m = ControlProtocolManager::instance ();
                for (list<ControlProtocolInfo*>::iterator i = m.control_protocol_info.begin(); i != m.control_protocol_info.end(); ++i) {
 
@@ -796,7 +835,7 @@ public:
 
 private:
 
-       void model_changed (TreeModel::Path const & p, TreeModel::iterator const & i)
+       void model_changed (TreeModel::Path const &, TreeModel::iterator const & i)
        {
                TreeModel::Row r = *i;
 
@@ -824,27 +863,62 @@ private:
                }
        }
 
+        void edit_clicked (GdkEventButton* ev)
+        {
+               if (ev->type != GDK_2BUTTON_PRESS) {
+                       return;
+               }
+
+               std::string name;
+               ControlProtocolInfo* cpi;
+               TreeModel::Row row;
+               
+               row = *(_view.get_selection()->get_selected());
+
+               Window* win = row[_model.editor];
+               if (win && !win->is_visible()) {
+                       win->present (); 
+               } else {
+                       cpi = row[_model.protocol_info];
+                       
+                       if (cpi && cpi->protocol && cpi->protocol->has_editor ()) {
+                               Box* box = (Box*) cpi->protocol->get_gui ();
+                               if (box) {
+                                       string title = row[_model.name];
+                                       ArdourDialog* win = new ArdourDialog (_parent, title);
+                                       win->get_vbox()->pack_start (*box, false, false);
+                                       box->show ();
+                                       win->present ();
+                                       row[_model.editor] = win;
+                               }
+                       }
+               }
+       }
+
         class ControlSurfacesModelColumns : public TreeModelColumnRecord
        {
        public:
-               
+
                ControlSurfacesModelColumns ()
                {
                        add (name);
                        add (enabled);
                        add (feedback);
                        add (protocol_info);
+                       add (editor);
                }
 
                TreeModelColumn<string> name;
                TreeModelColumn<bool> enabled;
                TreeModelColumn<bool> feedback;
                TreeModelColumn<ControlProtocolInfo*> protocol_info;
+               TreeModelColumn<Gtk::Window*> editor;
        };
 
        Glib::RefPtr<ListStore> _store;
        ControlSurfacesModelColumns _model;
        TreeView _view;
+        Gtk::Window& _parent;
 };
 
 
@@ -855,26 +929,26 @@ RCOptionEditor::RCOptionEditor ()
        /* MISC */
 
        add_option (_("Misc"), new OptionEditorHeading (_("Metering")));
-       
+
        ComboOption<float>* mht = new ComboOption<float> (
                "meter-hold",
                _("Meter hold time"),
-               mem_fun (*_rc_config, &RCConfiguration::get_meter_hold),
-               mem_fun (*_rc_config, &RCConfiguration::set_meter_hold)
+               sigc::mem_fun (*_rc_config, &RCConfiguration::get_meter_hold),
+               sigc::mem_fun (*_rc_config, &RCConfiguration::set_meter_hold)
                );
 
        mht->add (MeterHoldOff, _("off"));
        mht->add (MeterHoldShort, _("short"));
        mht->add (MeterHoldMedium, _("medium"));
        mht->add (MeterHoldLong, _("long"));
-       
+
        add_option (_("Misc"), mht);
 
        ComboOption<float>* mfo = new ComboOption<float> (
                "meter-falloff",
                _("Meter fall-off"),
-               mem_fun (*_rc_config, &RCConfiguration::get_meter_falloff),
-               mem_fun (*_rc_config, &RCConfiguration::set_meter_falloff)
+               sigc::mem_fun (*_rc_config, &RCConfiguration::get_meter_falloff),
+               sigc::mem_fun (*_rc_config, &RCConfiguration::set_meter_falloff)
                );
 
        mfo->add (METER_FALLOFF_OFF, _("off"));
@@ -888,62 +962,62 @@ RCOptionEditor::RCOptionEditor ()
        add_option (_("Misc"), mfo);
 
        add_option (_("Misc"), new OptionEditorHeading (_("Undo")));
-       
+
        add_option (_("Misc"), new UndoOptions (_rc_config));
 
        add_option (_("Misc"), new OptionEditorHeading (_("Misc")));
-       
+
 #ifndef GTKOSX
        /* font scaling does nothing with GDK/Quartz */
        add_option (_("Misc"), new FontScalingOptions (_rc_config));
-#endif 
+#endif
 
        add_option (_("Misc"),
             new BoolOption (
                     "verify-remove-last-capture",
                     _("Verify removal of last capture"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_verify_remove_last_capture),
-                    mem_fun (*_rc_config, &RCConfiguration::set_verify_remove_last_capture)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_verify_remove_last_capture),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_verify_remove_last_capture)
                     ));
-       
+
        add_option (_("Misc"),
             new BoolOption (
                     "periodic-safety-backups",
                     _("Make periodic backups of the session file"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_periodic_safety_backups),
-                    mem_fun (*_rc_config, &RCConfiguration::set_periodic_safety_backups)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_periodic_safety_backups),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_periodic_safety_backups)
                     ));
 
        add_option (_("Misc"),
             new BoolOption (
                     "sync-all-route-ordering",
                     _("Syncronise editor and mixer track order"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_sync_all_route_ordering),
-                    mem_fun (*_rc_config, &RCConfiguration::set_sync_all_route_ordering)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_sync_all_route_ordering),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_sync_all_route_ordering)
                     ));
 
        add_option (_("Misc"),
             new BoolOption (
                     "only-copy-imported-files",
                     _("Always copy imported files"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_only_copy_imported_files),
-                    mem_fun (*_rc_config, &RCConfiguration::set_only_copy_imported_files)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_only_copy_imported_files),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_only_copy_imported_files)
                     ));
 
        add_option (_("Misc"),
             new BoolOption (
                     "default-narrow_ms",
                     _("Use narrow mixer strips"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_default_narrow_ms),
-                    mem_fun (*_rc_config, &RCConfiguration::set_default_narrow_ms)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_default_narrow_ms),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_default_narrow_ms)
                     ));
 
        add_option (_("Misc"),
             new BoolOption (
                     "name-new-markers",
                     _("Name new markers"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_name_new_markers),
-                    mem_fun (*_rc_config, &RCConfiguration::set_name_new_markers)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_name_new_markers),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_name_new_markers)
                     ));
 
        /* TRANSPORT */
@@ -952,56 +1026,56 @@ RCOptionEditor::RCOptionEditor ()
             new BoolOption (
                     "latched-record-enable",
                     _("Keep record-enable engaged on stop"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_latched_record_enable),
-                    mem_fun (*_rc_config, &RCConfiguration::set_latched_record_enable)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_latched_record_enable),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_latched_record_enable)
                     ));
 
        add_option (_("Transport"),
             new BoolOption (
                     "stop-recording-on-xrun",
                     _("Stop recording when an xrun occurs"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_stop_recording_on_xrun),
-                    mem_fun (*_rc_config, &RCConfiguration::set_stop_recording_on_xrun)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_stop_recording_on_xrun),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_stop_recording_on_xrun)
                     ));
 
        add_option (_("Transport"),
             new BoolOption (
                     "create-xrun-marker",
                     _("Create markers where xruns occur"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_create_xrun_marker),
-                    mem_fun (*_rc_config, &RCConfiguration::set_create_xrun_marker)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_create_xrun_marker),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_create_xrun_marker)
                     ));
 
        add_option (_("Transport"),
             new BoolOption (
                     "stop-at-session-end",
                     _("Stop at the end of the session"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_stop_at_session_end),
-                    mem_fun (*_rc_config, &RCConfiguration::set_stop_at_session_end)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_stop_at_session_end),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_stop_at_session_end)
                     ));
 
        add_option (_("Transport"),
             new BoolOption (
                     "primary-clock-delta-edit-cursor",
                     _("Primary clock delta to edit cursor"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_primary_clock_delta_edit_cursor),
-                    mem_fun (*_rc_config, &RCConfiguration::set_primary_clock_delta_edit_cursor)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_primary_clock_delta_edit_cursor),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_primary_clock_delta_edit_cursor)
                     ));
 
        add_option (_("Transport"),
             new BoolOption (
                     "secondary-clock-delta-edit-cursor",
                     _("Secondary clock delta to edit cursor"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_secondary_clock_delta_edit_cursor),
-                    mem_fun (*_rc_config, &RCConfiguration::set_secondary_clock_delta_edit_cursor)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_secondary_clock_delta_edit_cursor),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_secondary_clock_delta_edit_cursor)
                     ));
 
        add_option (_("Transport"),
             new BoolOption (
                     "disable-disarm-during-roll",
-                    _("Disable record disarm when transport is rolling"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_disable_disarm_during_roll),
-                    mem_fun (*_rc_config, &RCConfiguration::set_disable_disarm_during_roll)
+                    _("Disable per-track record disarm while rolling"),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_disable_disarm_during_roll),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_disable_disarm_during_roll)
                     ));
 
        /* EDITOR */
@@ -1010,67 +1084,67 @@ RCOptionEditor::RCOptionEditor ()
             new BoolOption (
                     "link-region-and-track-selection",
                     _("Link selection of regions and tracks"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_link_region_and_track_selection),
-                    mem_fun (*_rc_config, &RCConfiguration::set_link_region_and_track_selection)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_link_region_and_track_selection),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_link_region_and_track_selection)
                     ));
 
        add_option (_("Editor"),
             new BoolOption (
                     "automation-follows-regions",
                     _("Move relevant automation when regions are moved"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_automation_follows_regions),
-                    mem_fun (*_rc_config, &RCConfiguration::set_automation_follows_regions)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_automation_follows_regions),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_automation_follows_regions)
                     ));
 
        add_option (_("Editor"),
             new BoolOption (
                     "show-track-meters",
                     _("Show meters on tracks in the editor"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_show_track_meters),
-                    mem_fun (*_rc_config, &RCConfiguration::set_show_track_meters)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_show_track_meters),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_show_track_meters)
                     ));
 
        add_option (_("Editor"),
             new BoolOption (
                     "use-overlap-equivalency",
                     _("Use overlap equivalency for regions"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_use_overlap_equivalency),
-                    mem_fun (*_rc_config, &RCConfiguration::set_use_overlap_equivalency)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_use_overlap_equivalency),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_use_overlap_equivalency)
                     ));
 
        add_option (_("Editor"),
             new BoolOption (
                     "rubberbanding-snaps-to-grid",
                     _("Make rubberband selection rectangle snap to the grid"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_rubberbanding_snaps_to_grid),
-                    mem_fun (*_rc_config, &RCConfiguration::set_rubberbanding_snaps_to_grid)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_rubberbanding_snaps_to_grid),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_rubberbanding_snaps_to_grid)
                     ));
 
        add_option (_("Editor"),
             new BoolOption (
                     "show-waveforms",
                     _("Show waveforms in regions"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_show_waveforms),
-                    mem_fun (*_rc_config, &RCConfiguration::set_show_waveforms)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_show_waveforms),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_show_waveforms)
                     ));
 
        ComboOption<WaveformScale>* wfs = new ComboOption<WaveformScale> (
                "waveform-scale",
                _("Waveform scale"),
-               mem_fun (*_rc_config, &RCConfiguration::get_waveform_scale),
-               mem_fun (*_rc_config, &RCConfiguration::set_waveform_scale)
+               sigc::mem_fun (*_rc_config, &RCConfiguration::get_waveform_scale),
+               sigc::mem_fun (*_rc_config, &RCConfiguration::set_waveform_scale)
                );
 
        wfs->add (Linear, _("linear"));
        wfs->add (Logarithmic, _("logarithmic"));
 
        add_option (_("Editor"), wfs);
-       
+
        ComboOption<WaveformShape>* wfsh = new ComboOption<WaveformShape> (
                "waveform-shape",
                _("Waveform shape"),
-               mem_fun (*_rc_config, &RCConfiguration::get_waveform_shape),
-               mem_fun (*_rc_config, &RCConfiguration::set_waveform_shape)
+               sigc::mem_fun (*_rc_config, &RCConfiguration::get_waveform_shape),
+               sigc::mem_fun (*_rc_config, &RCConfiguration::set_waveform_shape)
                );
 
        wfsh->add (Traditional, _("traditional"));
@@ -1082,40 +1156,50 @@ RCOptionEditor::RCOptionEditor ()
 
        add_option (_("Audio"), new OptionEditorHeading (_("Solo")));
 
-       ComboOption<SoloModel>* sm = new ComboOption<SoloModel> (
-               "solo-model",
-               _("Solo"),
-               mem_fun (*_rc_config, &RCConfiguration::get_solo_model),
-               mem_fun (*_rc_config, &RCConfiguration::set_solo_model)
+
+       add_option (_("Audio"),
+            new BoolOption (
+                    "solo-control-is-listen-control",
+                    _("Solo controls are Listen controls"),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_solo_control_is_listen_control),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_solo_control_is_listen_control)
+                    ));
+
+       ComboOption<ListenPosition>* lp = new ComboOption<ListenPosition> (
+               "listen-position",
+               _("Listen Position"),
+               sigc::mem_fun (*_rc_config, &RCConfiguration::get_listen_position),
+               sigc::mem_fun (*_rc_config, &RCConfiguration::set_listen_position)
                );
 
-       sm->add (SoloInPlace, _("in place"));
-       sm->add (SoloBus, _("via bus"));
+       lp->add (AfterFaderListen, _("after-fader listen"));
+       lp->add (PreFaderListen, _("pre-fader listen"));
 
-       add_option (_("Audio"), sm);
+       add_option (_("Audio"), lp);
+       add_option (_("Audio"), new SoloMuteOptions (_rc_config));
 
        add_option (_("Audio"),
             new BoolOption (
                     "solo-latched",
                     _("Latched solo"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_solo_latched),
-                    mem_fun (*_rc_config, &RCConfiguration::set_solo_latched)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_solo_latched),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_solo_latched)
                     ));
 
        add_option (_("Audio"),
             new BoolOption (
                     "show-solo-mutes",
                     _("Show solo muting"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_show_solo_mutes),
-                    mem_fun (*_rc_config, &RCConfiguration::set_show_solo_mutes)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_show_solo_mutes),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_show_solo_mutes)
                     ));
 
        add_option (_("Audio"),
             new BoolOption (
                     "solo-mute-override",
                     _("Override muting"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_solo_mute_override),
-                    mem_fun (*_rc_config, &RCConfiguration::set_solo_mute_override)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_solo_mute_override),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_solo_mute_override)
                     ));
 
        add_option (_("Audio"), new OptionEditorHeading (_("Monitoring")));
@@ -1123,8 +1207,8 @@ RCOptionEditor::RCOptionEditor ()
        ComboOption<MonitorModel>* mm = new ComboOption<MonitorModel> (
                "monitoring-model",
                _("Monitoring handled by"),
-               mem_fun (*_rc_config, &RCConfiguration::get_monitoring_model),
-               mem_fun (*_rc_config, &RCConfiguration::set_monitoring_model)
+               sigc::mem_fun (*_rc_config, &RCConfiguration::get_monitoring_model),
+               sigc::mem_fun (*_rc_config, &RCConfiguration::set_monitoring_model)
                );
 
        mm->add (HardwareMonitoring, _("JACK"));
@@ -1137,8 +1221,8 @@ RCOptionEditor::RCOptionEditor ()
             new BoolOption (
                     "tape-machine-mode",
                     _("Tape machine mode"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_tape_machine_mode),
-                    mem_fun (*_rc_config, &RCConfiguration::set_tape_machine_mode)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_tape_machine_mode),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_tape_machine_mode)
                     ));
 
        add_option (_("Audio"), new OptionEditorHeading (_("Connection of tracks and busses")));
@@ -1147,15 +1231,15 @@ RCOptionEditor::RCOptionEditor ()
                    new BoolOption (
                            "auto-connect-standard-busses",
                            _("Auto-connect master/monitor busses"),
-                           mem_fun (*_rc_config, &RCConfiguration::get_auto_connect_standard_busses),
-                           mem_fun (*_rc_config, &RCConfiguration::set_auto_connect_standard_busses)
+                           sigc::mem_fun (*_rc_config, &RCConfiguration::get_auto_connect_standard_busses),
+                           sigc::mem_fun (*_rc_config, &RCConfiguration::set_auto_connect_standard_busses)
                            ));
 
        ComboOption<AutoConnectOption>* iac = new ComboOption<AutoConnectOption> (
                "input-auto-connect",
                _("Connect track and bus inputs"),
-               mem_fun (*_rc_config, &RCConfiguration::get_input_auto_connect),
-               mem_fun (*_rc_config, &RCConfiguration::set_input_auto_connect)
+               sigc::mem_fun (*_rc_config, &RCConfiguration::get_input_auto_connect),
+               sigc::mem_fun (*_rc_config, &RCConfiguration::set_input_auto_connect)
                );
 
        iac->add (AutoConnectPhysical, _("automatically to physical inputs"));
@@ -1166,8 +1250,8 @@ RCOptionEditor::RCOptionEditor ()
        ComboOption<AutoConnectOption>* oac = new ComboOption<AutoConnectOption> (
                "output-auto-connect",
                _("Connect track and bus outputs"),
-               mem_fun (*_rc_config, &RCConfiguration::get_output_auto_connect),
-               mem_fun (*_rc_config, &RCConfiguration::set_output_auto_connect)
+               sigc::mem_fun (*_rc_config, &RCConfiguration::get_output_auto_connect),
+               sigc::mem_fun (*_rc_config, &RCConfiguration::set_output_auto_connect)
                );
 
        oac->add (AutoConnectPhysical, _("automatically to physical outputs"));
@@ -1182,33 +1266,33 @@ RCOptionEditor::RCOptionEditor ()
             new BoolOption (
                     "denormal-protection",
                     _("Use DC bias to protect against denormals"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_denormal_protection),
-                    mem_fun (*_rc_config, &RCConfiguration::set_denormal_protection)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_denormal_protection),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_denormal_protection)
                     ));
 
        ComboOption<DenormalModel>* dm = new ComboOption<DenormalModel> (
                "denormal-model",
                _("Processor handling"),
-               mem_fun (*_rc_config, &RCConfiguration::get_denormal_model),
-               mem_fun (*_rc_config, &RCConfiguration::set_denormal_model)
+               sigc::mem_fun (*_rc_config, &RCConfiguration::get_denormal_model),
+               sigc::mem_fun (*_rc_config, &RCConfiguration::set_denormal_model)
                );
-       
+
        dm->add (DenormalNone, _("no processor handling"));
-       
+
        FPU fpu;
-       
+
        if (fpu.has_flush_to_zero()) {
                dm->add (DenormalFTZ, _("use FlushToZero"));
        }
-       
+
        if (fpu.has_denormals_are_zero()) {
                dm->add (DenormalDAZ, _("use DenormalsAreZero"));
        }
-       
+
        if (fpu.has_flush_to_zero() && fpu.has_denormals_are_zero()) {
                dm->add (DenormalFTZDAZ, _("use FlushToZero and DenormalsAreZerO"));
        }
-       
+
        add_option (_("Audio"), dm);
 
        add_option (_("Audio"), new OptionEditorHeading (_("Plugins")));
@@ -1217,44 +1301,111 @@ RCOptionEditor::RCOptionEditor ()
             new BoolOption (
                     "plugins-stop-with-transport",
                     _("Stop plugins when the transport is stopped"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_plugins_stop_with_transport),
-                    mem_fun (*_rc_config, &RCConfiguration::set_plugins_stop_with_transport)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_plugins_stop_with_transport),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_plugins_stop_with_transport)
                     ));
 
        add_option (_("Audio"),
             new BoolOption (
                     "do-not-record-plugins",
                     _("Disable plugins during recording"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_do_not_record_plugins),
-                    mem_fun (*_rc_config, &RCConfiguration::set_do_not_record_plugins)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_do_not_record_plugins),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_do_not_record_plugins)
                     ));
 
        add_option (_("Audio"),
             new BoolOption (
                     "new-plugins-active",
                     _("Make new plugins active"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_new_plugins_active),
-                    mem_fun (*_rc_config, &RCConfiguration::set_new_plugins_active)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_new_plugins_active),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_new_plugins_active)
                     ));
 
        add_option (_("Audio"),
             new BoolOption (
                     "auto-analyse-audio",
                     _("Enable automatic analysis of audio"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_auto_analyse_audio),
-                    mem_fun (*_rc_config, &RCConfiguration::set_auto_analyse_audio)
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_auto_analyse_audio),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_auto_analyse_audio)
                     ));
 
        /* MIDI CONTROL */
 
-       add_option (_("MIDI control"), new MIDIPorts (_rc_config));
+       list<ComboOption<string>* > midi_combos;
+
+       midi_combos.push_back (new ComboOption<string> (
+                                      "mtc-port-name",
+                                      _("Send/Receive MTC via"),
+                                      sigc::mem_fun (*_rc_config, &RCConfiguration::get_mtc_port_name),
+                                      sigc::mem_fun (*_rc_config, &RCConfiguration::set_mtc_port_name)
+                                      ));
+
+       midi_combos.push_back (new ComboOption<string> (
+                                      "midi-clock-port-name",
+                                      _("Send/Receive MIDI clock via"),
+                                      sigc::mem_fun (*_rc_config, &RCConfiguration::get_midi_clock_port_name),
+                                      sigc::mem_fun (*_rc_config, &RCConfiguration::set_midi_clock_port_name)
+                                      ));
+
+       midi_combos.push_back (new ComboOption<string> (
+                                      "mmc-port-name",
+                                      _("Send/Receive MMC via"),
+                                      sigc::mem_fun (*_rc_config, &RCConfiguration::get_mmc_port_name),
+                                      sigc::mem_fun (*_rc_config, &RCConfiguration::set_mmc_port_name)
+                                      ));
+
+       midi_combos.push_back (new ComboOption<string> (
+                                      "midi-port-name",
+                                      _("Send/Receive MIDI parameter control via"),
+                                      sigc::mem_fun (*_rc_config, &RCConfiguration::get_midi_port_name),
+                                      sigc::mem_fun (*_rc_config, &RCConfiguration::set_midi_port_name)
+                                      ));
+       
+       add_option (_("MIDI control"), new MIDIPorts (_rc_config, midi_combos));
+
+       for (list<ComboOption<string>* >::iterator i = midi_combos.begin(); i != midi_combos.end(); ++i) {
+               add_option (_("MIDI control"), *i);
+       }
+
+       add_option (_("MIDI control"),
+                   new BoolOption (
+                           "send-mtc",
+                           _("Send MIDI Time Code"),
+                           sigc::mem_fun (*_rc_config, &RCConfiguration::get_send_mtc),
+                           sigc::mem_fun (*_rc_config, &RCConfiguration::set_send_mtc)
+                           ));
+
+       add_option (_("MIDI control"),
+                   new BoolOption (
+                           "mmc-control",
+                           _("Obey MIDI Machine Control commands"),
+                           sigc::mem_fun (*_rc_config, &RCConfiguration::get_mmc_control),
+                           sigc::mem_fun (*_rc_config, &RCConfiguration::set_mmc_control)
+                           ));
+
+
+       add_option (_("MIDI control"),
+                   new BoolOption (
+                           "send-mmc",
+                           _("Send MIDI Machine Control commands"),
+                           sigc::mem_fun (*_rc_config, &RCConfiguration::get_send_mmc),
+                           sigc::mem_fun (*_rc_config, &RCConfiguration::set_send_mmc)
+                           ));
+
+       add_option (_("MIDI control"),
+                   new BoolOption (
+                           "midi-feedback",
+                           _("Send MIDI control feedback"),
+                           sigc::mem_fun (*_rc_config, &RCConfiguration::get_midi_feedback),
+                           sigc::mem_fun (*_rc_config, &RCConfiguration::set_midi_feedback)
+                           ));
 
        add_option (_("MIDI control"),
             new SpinOption<uint8_t> (
                     "mmc-receive-device-id",
                     _("Inbound MMC device ID"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_mmc_receive_device_id),
-                    mem_fun (*_rc_config, &RCConfiguration::set_mmc_receive_device_id),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_mmc_receive_device_id),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_mmc_receive_device_id),
                     0, 128, 1, 10
                     ));
 
@@ -1262,8 +1413,8 @@ RCOptionEditor::RCOptionEditor ()
             new SpinOption<uint8_t> (
                     "mmc-send-device-id",
                     _("Outbound MMC device ID"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_mmc_send_device_id),
-                    mem_fun (*_rc_config, &RCConfiguration::set_mmc_send_device_id),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_mmc_send_device_id),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_mmc_send_device_id),
                     0, 128, 1, 10
                     ));
 
@@ -1271,20 +1422,20 @@ RCOptionEditor::RCOptionEditor ()
             new SpinOption<int32_t> (
                     "initial-program-change",
                     _("Initial program change"),
-                    mem_fun (*_rc_config, &RCConfiguration::get_initial_program_change),
-                    mem_fun (*_rc_config, &RCConfiguration::set_initial_program_change),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::get_initial_program_change),
+                    sigc::mem_fun (*_rc_config, &RCConfiguration::set_initial_program_change),
                     -1, 65536, 1, 10
                     ));
 
        /* CONTROL SURFACES */
 
-       add_option (_("Control surfaces"), new ControlSurfacesOptions);
+       add_option (_("Control surfaces"), new ControlSurfacesOptions (*this));
 
        ComboOption<RemoteModel>* rm = new ComboOption<RemoteModel> (
                "remote-model",
                _("Control surface remote ID"),
-               mem_fun (*_rc_config, &RCConfiguration::get_remote_model),
-               mem_fun (*_rc_config, &RCConfiguration::set_remote_model)
+               sigc::mem_fun (*_rc_config, &RCConfiguration::get_remote_model),
+               sigc::mem_fun (*_rc_config, &RCConfiguration::set_remote_model)
                );
 
        rm->add (UserOrdered, _("assigned by user"));