From 7886d3232ca2eafedc05e12bd0adbd5abe535f64 Mon Sep 17 00:00:00 2001
From: Taybin Rutkin <taybin@taybin.com>
Date: Tue, 29 Nov 2005 15:10:41 +0000
Subject: [PATCH] Added comment explaining new Gtk::Dialog idiom.

git-svn-id: svn://localhost/trunk/ardour2@148 d708f5d6-7413-0410-9779-e7cbd77b26cf
---
 gtk2_ardour/ardour_ui_dialogs.cc | 21 ++++++++++++++-------
 1 file changed, 14 insertions(+), 7 deletions(-)

diff --git a/gtk2_ardour/ardour_ui_dialogs.cc b/gtk2_ardour/ardour_ui_dialogs.cc
index aa06e25ad5..3d299e9ac2 100644
--- a/gtk2_ardour/ardour_ui_dialogs.cc
+++ b/gtk2_ardour/ardour_ui_dialogs.cc
@@ -333,13 +333,20 @@ ARDOUR_UI::route_params_hiding ()
 void
 ARDOUR_UI::toggle_sound_file_browser ()
 {
-	// GTK2FIX
-	// if (sfdb_check->get_active()) {
-	// SoundFileBrowser sfdb(_("Sound File Browser"));
+	/* This is called from the check menu item.  If checked on, open
+	 * a new SoundFileBrowser, and connect it's quit method to the
+	 * check menu item so if it is toggled off, it exits. If it exits
+	 * by itself, set the check menu item to false.
+	 * If this is called by checking off, don't do anything, the signals
+	 * should handle everything.  I expect this idiom to be useful for 
+	 * other Gtk::Dialog's as well.  --Taybin */
 
-	// sfdb_check->signal_toggled().connect (bind (mem_fun (sfdb, &Gtk::Dialog::response), Gtk::RESPONSE_CANCEL));
-	// sfdb.run();
-	// sfdb_check->set_active(false);
-	// }
+	//GTK2FIX
+	//if (sfdb_check->get_active()) {
+	//SoundFileBrowser sfdb(_("Sound File Browser"));
+	//sfdb_check->signal_toggled().connect (bind (mem_fun (sfdb, &Gtk::Dialog::response), Gtk::RESPONSE_CANCEL));
+	//sfdb.run();
+	//sfdb_check->set_active(false);
+	//}
 }