X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Fwx%2Fhints_dialog.cc;h=36c9eb6c8e54f4ddefc311a19b5620d124c58a04;hb=e60bb3e51bd1508b149e6b8f6608f09b5196ae26;hp=9f89569481f0d8f17cd8e570b68216d0d768a7a8;hpb=9980c40a47abe24f8dd3cb16e14ad4a3254a8465;p=dcpomatic.git

diff --git a/src/wx/hints_dialog.cc b/src/wx/hints_dialog.cc
index 9f8956948..36c9eb6c8 100644
--- a/src/wx/hints_dialog.cc
+++ b/src/wx/hints_dialog.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2012-2013 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2012-2015 Carl Hetherington <cth@carlh.net>
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
@@ -17,18 +17,22 @@
 
 */
 
-#include <boost/algorithm/string.hpp>
-#include <wx/richtext/richtextctrl.h>
 #include "lib/film.h"
 #include "lib/ratio.h"
+#include "lib/video_content.h"
+#include "lib/subtitle_content.h"
+#include "lib/font.h"
 #include "hints_dialog.h"
+#include <wx/richtext/richtextctrl.h>
+#include <boost/algorithm/string.hpp>
+#include <boost/foreach.hpp>
 
 using boost::shared_ptr;
 using boost::dynamic_pointer_cast;
 
-HintsDialog::HintsDialog (wxWindow* parent, boost::weak_ptr<Film> f)
+HintsDialog::HintsDialog (wxWindow* parent, boost::weak_ptr<Film> film)
 	: wxDialog (parent, wxID_ANY, _("Hints"))
-	, _film (f)
+	, _film (film)
 {
 	wxBoxSizer* sizer = new wxBoxSizer (wxVERTICAL);
 	_text = new wxRichTextCtrl (this, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (400, 300), wxRE_READONLY);
@@ -45,10 +49,10 @@ HintsDialog::HintsDialog (wxWindow* parent, boost::weak_ptr<Film> f)
 
 	_text->GetCaret()->Hide ();
 
-	boost::shared_ptr<Film> film = _film.lock ();
-	if (film) {
-		film->Changed.connect (boost::bind (&HintsDialog::film_changed, this));
-		film->ContentChanged.connect (boost::bind (&HintsDialog::film_changed, this));
+	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_changed ();
@@ -59,13 +63,35 @@ HintsDialog::film_changed ()
 {
 	_text->Clear ();
 	bool hint = false;
-	
+
 	boost::shared_ptr<Film> film = _film.lock ();
 	if (!film) {
 		return;
 	}
 
+	ContentList content = film->content ();
+
 	_text->BeginStandardBullet (N_("standard/circle"), 1, 50);
+
+	bool big_font_files = false;
+	if (film->interop ()) {
+		BOOST_FOREACH (shared_ptr<Content> i, content) {
+			shared_ptr<SubtitleContent> s = dynamic_pointer_cast<SubtitleContent> (i);
+			if (s) {
+				BOOST_FOREACH (shared_ptr<Font> j, s->fonts ()) {
+					if (j->file() && boost::filesystem::file_size (j->file().get ()) >= (640 * 1024)) {
+						big_font_files = true;
+					}
+				}
+			}
+		}
+	}
+
+	if (big_font_files) {
+		hint = true;
+		_text->WriteText (_("You have specified a font file which is larger than 640kB.  This is very likely to cause problems on playback."));
+	}
+
 	if (film->audio_channels() % 2) {
 		hint = true;
 		_text->WriteText (_("Your DCP has an odd number of audio channels.  This is very likely to cause problems on playback."));
@@ -81,7 +107,6 @@ HintsDialog::film_changed ()
 		_text->Newline ();
 	}
 
-	ContentList content = film->content ();
 	int flat_or_narrower = 0;
 	int scope = 0;
 	for (ContentList::const_iterator i = content.begin(); i != content.end(); ++i) {
@@ -107,7 +132,7 @@ HintsDialog::film_changed ()
 		_text->WriteText (_("All of your content is at 1.85:1 or narrower but your DCP's container is Scope (2.39:1).  This will pillar-box your content inside a Flat (1.85:1) frame.  You may prefer to set your DCP's container to Flat (1.85:1) in the \"DCP\" tab."));
 		_text->Newline ();
 	}
-	
+
 	if (film->video_frame_rate() != 24 && film->video_frame_rate() != 48) {
 		hint = true;
 		_text->WriteText (wxString::Format (_("Your DCP frame rate (%d fps) may cause problems in a few (mostly older) projectors.  Use 24 or 48 frames per second to be on the safe side."), film->video_frame_rate()));