Replace May/Done/NotDone signal sets with one signal and extend

this treatment to anything that caused Player::setup_pieces.  This should
fix out-of-sequence Player emissions caused by setup_pieces being called
by one thread while the butler is calling pass().

1 files changed, 9 insertions, 1 deletions

diff --git a/src/wx/hints_dialog.cc b/src/wx/hints_dialog.cc
index 2c02a4f72..3872ea8ae 100644
--- a/src/wx/hints_dialog.cc
+++ b/src/wx/hints_dialog.cc
@@ -76,7 +76,7 @@ HintsDialog::HintsDialog (wxWindow* parent, boost::weak_ptr<Film> film, bool ok)
 	boost::shared_ptr<Film> locked_film = _film.lock ();
 	if (locked_film) {
 		_film_changed_connection = locked_film->Changed.connect (boost::bind (&HintsDialog::film_changed, this));
-		_film_content_changed_connection = locked_film->ContentChanged.connect (boost::bind (&HintsDialog::film_changed, this));
+		_film_content_changed_connection = locked_film->ContentChange.connect (boost::bind (&HintsDialog::film_content_change, this, _1));
 	}
 
 	_hints->Hint.connect (bind (&HintsDialog::hint, this, _1));
@@ -107,6 +107,14 @@ HintsDialog::film_changed ()
 }
 
 void
+HintsDialog::film_content_change (ChangeType type)
+{
+	if (type == CHANGE_TYPE_DONE) {
+		film_changed ();
+	}
+}
+
+void
 HintsDialog::update ()
 {
 	_text->Clear ();