fix incorrect accumulation of export video options each time the dialog is used

[ardour.git] / gtk2_ardour / transport_masters_dialog.cc

diff --git a/gtk2_ardour/transport_masters_dialog.cc b/gtk2_ardour/transport_masters_dialog.cc
index 3cb7233f17eeea5b3f3609d4ed3ed5db58ca4f74..33d0ce5ca4fa0b4ba7c0e823f494bc7fa1cd29ba 100644 (file)
--- a/gtk2_ardour/transport_masters_dialog.cc
+++ b/gtk2_ardour/transport_masters_dialog.cc
@@ -134,7 +134,12 @@ TransportMastersWidget::add_master ()
                }
        }
 
-       TransportMasterManager::instance().add (d.get_type(), name);
+       d.hide ();
+
+       if (TransportMasterManager::instance().add (d.get_type(), name)) {
+               MessageDialog msg (_("New transport master not added - check error log for details"));
+               msg.run ();
+       }
 }
 
 void
@@ -208,7 +213,6 @@ TransportMastersWidget::rebuild ()
 
                table.show_all ();
 
-               // r->label_box.set_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK);
                r->label_box.signal_button_press_event().connect (sigc::mem_fun (*r, &TransportMastersWidget::Row::name_press));
                r->port_combo.signal_changed().connect (sigc::mem_fun (*r, &TransportMastersWidget::Row::port_choice_changed));
                r->use_button.signal_toggled().connect (sigc::mem_fun (*r, &TransportMastersWidget::Row::use_button_toggled));
@@ -262,13 +266,11 @@ TransportMastersWidget::Row::name_press (GdkEventButton* ev)
        return false;
 }
 
-gboolean
-TransportMastersWidget::Row::_idle_remove (gpointer arg)
+bool
+TransportMastersWidget::idle_remove (TransportMastersWidget::Row* row)
 {
-       TransportMastersWidget::Row* row = (TransportMastersWidget::Row*) arg;
        TransportMasterManager::instance().remove (row->tm->name());
-
-       return FALSE; /* do not call again */
+       return false;
 }
 
 void
@@ -277,7 +279,7 @@ TransportMastersWidget::Row::remove_clicked ()
        /* have to do this via an idle callback, because it will destroy the
           widget from which this callback was initiated.
        */
-       g_idle_add_full (G_PRIORITY_HIGH_IDLE + 10, _idle_remove, this, NULL);
+       Glib::signal_idle().connect (sigc::bind (sigc::mem_fun (parent, &TransportMastersWidget::idle_remove), this));
 }
 
 void