From: Carl Hetherington <cth@carlh.net>
Date: Sun, 16 Jan 2022 14:18:55 +0000 (+0100)
Subject: Set window title based on the directory from Film.
X-Git-Tag: v2.16.10~19
X-Git-Url: https://git.carlh.net/gitweb/?p=dcpomatic.git;a=commitdiff_plain;h=3cbc5434edfd0eff96d4e25313d1dfcde478070c

Set window title based on the directory from Film.
---

diff --git a/src/tools/dcpomatic.cc b/src/tools/dcpomatic.cc
index 1ea9ab63b..a14a27343 100644
--- a/src/tools/dcpomatic.cc
+++ b/src/tools/dcpomatic.cc
@@ -368,9 +368,8 @@ public:
 
 		set_menu_sensitivity ();
 
-		_film_editor->FileChanged.connect (bind (&DOMFrame::file_changed, this, _1));
 		_film_editor->content_panel()->SelectionChanged.connect (boost::bind (&DOMFrame::set_menu_sensitivity, this));
-		file_changed ("");
+		set_title ();
 
 		JobManager::instance()->ActiveJobsChanged.connect (boost::bind (&DOMFrame::set_menu_sensitivity, this));
 
@@ -498,6 +497,7 @@ public:
 			_film->Message.connect (boost::bind(&DOMFrame::film_message, this, _1));
 			dcpomatic_log = _film->log ();
 		}
+		set_title ();
 	}
 
 	shared_ptr<Film> film () const {
@@ -526,16 +526,6 @@ private:
 		}
 	}
 
-	void file_changed (boost::filesystem::path f)
-	{
-		auto s = wx_to_std(_("DCP-o-matic"));
-		if (!f.empty ()) {
-			s += " - " + f.string ();
-		}
-
-		SetTitle (std_to_wx (s));
-	}
-
 	void file_new ()
 	{
 		auto d = new FilmNameLocationDialog (this, _("New Film"), true);
@@ -1535,6 +1525,16 @@ private:
 		d->Destroy();
 	}
 
+	void set_title ()
+	{
+		auto s = wx_to_std(_("DCP-o-matic"));
+		if (_film && _film->directory()) {
+			s += " - " + _film->directory()->string();
+		}
+
+		SetTitle (std_to_wx(s));
+	}
+
 	FilmEditor* _film_editor;
 	std::shared_ptr<FilmViewer> _film_viewer;
 	StandardControls* _controls;
diff --git a/src/wx/film_editor.cc b/src/wx/film_editor.cc
index 47fb46970..2ec174260 100644
--- a/src/wx/film_editor.cc
+++ b/src/wx/film_editor.cc
@@ -135,19 +135,12 @@ FilmEditor::set_film (shared_ptr<Film> film)
 	_dcp_panel->set_film (_film);
 
 	if (!_film) {
-		FileChanged ("");
 		return;
 	}
 
 	_film->Change.connect (bind(&FilmEditor::film_change, this, _1, _2));
 	_film->ContentChange.connect (bind(&FilmEditor::film_content_change, this, _1, _3));
 
-	if (_film->directory()) {
-		FileChanged (_film->directory().get());
-	} else {
-		FileChanged ("");
-	}
-
 	if (!_film->content().empty()) {
 		_content_panel->set_selection (_film->content().front());
 	}
diff --git a/src/wx/film_editor.h b/src/wx/film_editor.h
index 61557ce1c..0ba021b62 100644
--- a/src/wx/film_editor.h
+++ b/src/wx/film_editor.h
@@ -50,7 +50,6 @@ public:
 	void set_film (std::shared_ptr<Film>);
 	void first_shown ();
 
-	boost::signals2::signal<void (boost::filesystem::path)> FileChanged;
 	boost::signals2::signal<void (void)> SelectionChanged;
 
 	/* Stuff for panels */