From dd9f4f7e9511f8f830ec05d1b60c475c6b2d71e0 Mon Sep 17 00:00:00 2001
From: Carl Hetherington <cth@carlh.net>
Date: Fri, 27 Apr 2018 15:13:42 +0100
Subject: Fix incorrect container size when loading a VF/OV combination into
 the player.

---
 src/tools/dcpomatic_player.cc | 30 ++++++++++++++++++------------
 1 file changed, 18 insertions(+), 12 deletions(-)

(limited to 'src/tools')

diff --git a/src/tools/dcpomatic_player.cc b/src/tools/dcpomatic_player.cc
index 19768e292..2fe9aca8b 100644
--- a/src/tools/dcpomatic_player.cc
+++ b/src/tools/dcpomatic_player.cc
@@ -213,19 +213,12 @@ public:
 			return;
 		}
 
-		if (dcp->subtitle) {
-			dcp->subtitle->set_use (true);
-		}
+		setup_from_dcp (dcp);
 
 		if (dcp->three_d()) {
 			_film->set_three_d (true);
 		}
 
-		Ratio const * r = Ratio::nearest_from_ratio(dcp->video->size().ratio());
-		if (r) {
-			_film->set_container(r);
-		}
-
 		_viewer->set_film (_film);
 		_viewer->set_position (DCPTime ());
 		_info->triggered_update ();
@@ -364,10 +357,7 @@ private:
 			DCPOMATIC_ASSERT (dcp);
 			dcp->add_ov (wx_to_std(c->GetPath()));
 			dcp->examine (shared_ptr<Job>());
-			/* Maybe we just gained some subtitles */
-			if (dcp->subtitle) {
-				dcp->subtitle->set_use (true);
-			}
+			setup_from_dcp (dcp);
 		}
 
 		c->Destroy ();
@@ -594,6 +584,22 @@ private:
 		_viewer->forward_frame ();
 	}
 
+private:
+
+	void setup_from_dcp (shared_ptr<DCPContent> dcp)
+	{
+		if (dcp->subtitle) {
+			dcp->subtitle->set_use (true);
+		}
+
+		if (dcp->video) {
+			Ratio const * r = Ratio::nearest_from_ratio(dcp->video->size().ratio());
+			if (r) {
+				_film->set_container(r);
+			}
+		}
+	}
+
 	bool _update_news_requested;
 	PlayerInformation* _info;
 	wxPreferencesEditor* _config_dialog;
-- 
cgit v1.2.3