Add hint about > 640kB font files (#588).

[dcpomatic.git] / src / wx / hints_dialog.cc

diff --git a/src/wx/hints_dialog.cc b/src/wx/hints_dialog.cc
index 9f89569481f0d8f17cd8e570b68216d0d768a7a8..83b4ece84ddbe639d201fc971f11855b9bf8040f 100644 (file)
--- 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,11 +17,15 @@
 
 */
 
-#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;
@@ -47,8 +51,8 @@ HintsDialog::HintsDialog (wxWindow* parent, boost::weak_ptr<Film> f)
 
        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));
+               _film_changed_connection = film->Changed.connect (boost::bind (&HintsDialog::film_changed, this));
+               _film_content_changed_connection = film->ContentChanged.connect (boost::bind (&HintsDialog::film_changed, this));
        }
 
        film_changed ();
@@ -65,7 +69,29 @@ HintsDialog::film_changed ()
                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) {