Take Film pointer out of Content.
[dcpomatic.git] / src / tools / dcpomatic_player.cc
index 5dcd331ca3bca2346b00b354909d69bd486e7ee1..7028624a847116c270df7839a248f0781485ecfb 100644 (file)
@@ -38,6 +38,7 @@
 #include "lib/job_manager.h"
 #include "lib/job.h"
 #include "lib/film.h"
+#include "lib/null_log.h"
 #include "lib/video_content.h"
 #include "lib/text_content.h"
 #include "lib/ratio.h"
 #include "lib/server.h"
 #include "lib/dcpomatic_socket.h"
 #include "lib/scoped_temporary.h"
+#include "lib/monitor_checker.h"
+#include "lib/lock_file_checker.h"
+#include "lib/ffmpeg_content.h"
+#include <dcp/dcp.h>
+#include <dcp/raw_convert.h>
+#include <dcp/exceptions.h>
 #include <wx/wx.h>
 #include <wx/stdpaths.h>
 #include <wx/splash.h>
@@ -120,6 +127,9 @@ public:
                , _view_full_screen (0)
                , _view_dual_screen (0)
        {
+               /* XXX */
+               dcpomatic_log.reset (new NullLog());
+               cout << "here's the log " << dcpomatic_log.get() << "\n";
 
 #if defined(DCPOMATIC_WINDOWS)
                maybe_open_console ();
@@ -135,8 +145,8 @@ public:
                SetIcon (wxIcon (std_to_wx ("id")));
 #endif
 
-               _config_changed_connection = Config::instance()->Changed.connect (boost::bind (&DOMFrame::config_changed, this));
-               config_changed ();
+               _config_changed_connection = Config::instance()->Changed.connect (boost::bind (&DOMFrame::config_changed, this, _1));
+               update_from_config ();
 
                Bind (wxEVT_MENU, boost::bind (&DOMFrame::file_open, this), ID_file_open);
                Bind (wxEVT_MENU, boost::bind (&DOMFrame::file_add_ov, this), ID_file_add_ov);
@@ -169,8 +179,11 @@ public:
                _viewer->Started.connect (bind(&DOMFrame::playback_started, this, _1));
                _viewer->Seeked.connect (bind(&DOMFrame::playback_seeked, this, _1));
                _viewer->Stopped.connect (bind(&DOMFrame::playback_stopped, this, _1));
+#ifdef DCPOMATIC_VARIANT_SWAROOP
+               _viewer->PositionChanged.connect (bind(&DOMFrame::position_changed, this));
+#endif
                _info = new PlayerInformation (_overall_panel, _viewer);
-               setup_main_sizer (true);
+               setup_main_sizer (Config::instance()->player_mode());
 #ifdef __WXOSX__
                int accelerators = 4;
 #else
@@ -192,20 +205,77 @@ public:
                Bind (wxEVT_MENU, boost::bind (&DOMFrame::back_frame, this), ID_back_frame);
                Bind (wxEVT_MENU, boost::bind (&DOMFrame::forward_frame, this), ID_forward_frame);
 
-               UpdateChecker::instance()->StateChanged.connect (boost::bind (&DOMFrame::update_checker_state_changed, this));
-               _controls->DCPDirectorySelected.connect (boost::bind(&DOMFrame::load_dcp, this, _1));
-               _controls->DCPEjected.connect (boost::bind(&DOMFrame::eject_dcp, this));
+               reset_film ();
 
+               UpdateChecker::instance()->StateChanged.connect (boost::bind (&DOMFrame::update_checker_state_changed, this));
+#ifdef DCPOMATIC_VARIANT_SWAROOP
+               MonitorChecker::instance()->StateChanged.connect(boost::bind(&DOMFrame::monitor_checker_state_changed, this));
+               MonitorChecker::instance()->run ();
+               LockFileChecker::instance()->StateChanged.connect(boost::bind(&DOMFrame::lock_checker_state_changed, this));
+               LockFileChecker::instance()->run ();
+#endif
                setup_screen ();
+
+#ifdef DCPOMATIC_VARIANT_SWAROOP
+               if (
+                       boost::filesystem::is_regular_file(Config::path("position")) &&
+                       boost::filesystem::is_regular_file(Config::path("spl.xml"))) {
+
+                       shared_ptr<Film> film (new Film(boost::optional<boost::filesystem::path>()));
+                       film->read_metadata (Config::path("spl.xml"));
+                       reset_film (film);
+                       FILE* f = fopen_boost (Config::path("position"), "r");
+                       if (f) {
+                               char buffer[64];
+                               fscanf (f, "%63s", buffer);
+                               _viewer->seek (DCPTime(atoi(buffer)), true);
+                               _viewer->start ();
+                               fclose (f);
+                       }
+               }
+
+#endif
+       }
+
+       void position_changed ()
+       {
+               if (!_viewer->playing() || _viewer->position().get() % DCPTime::HZ) {
+                       return;
+               }
+
+               FILE* f = fopen_boost (Config::path("position"), "w");
+               if (f) {
+                       string const p = dcp::raw_convert<string> (_viewer->position().get());
+                       fwrite (p.c_str(), p.length(), 1, f);
+                       fclose (f);
+               }
        }
 
-       void setup_main_sizer (bool with_viewer)
+#ifdef DCPOMATIC_VARIANT_SWAROOP
+       void monitor_checker_state_changed ()
+       {
+               if (!MonitorChecker::instance()->ok()) {
+                       _viewer->stop ();
+                       error_dialog (this, _("The required display devices are not connected correctly."));
+               }
+       }
+
+       void lock_checker_state_changed ()
+       {
+               if (!LockFileChecker::instance()->ok()) {
+                       _viewer->stop ();
+                       error_dialog (this, _("The lock file is not present."));
+               }
+       }
+#endif
+
+       void setup_main_sizer (Config::PlayerMode mode)
        {
                wxSizer* main_sizer = new wxBoxSizer (wxVERTICAL);
-               if (with_viewer) {
+               if (mode != Config::PLAYER_MODE_DUAL) {
                        main_sizer->Add (_viewer->panel(), 1, wxEXPAND | wxALIGN_CENTER_VERTICAL);
                }
-               main_sizer->Add (_controls, 0, wxEXPAND | wxALL, 6);
+               main_sizer->Add (_controls, mode == Config::PLAYER_MODE_DUAL ? 1 : 0, wxEXPAND | wxALL, 6);
                main_sizer->Add (_info, 0, wxEXPAND | wxALL, 6);
                _overall_panel->SetSizer (main_sizer);
                _overall_panel->Layout ();
@@ -213,6 +283,16 @@ public:
 
        bool playback_permitted ()
        {
+#ifdef DCPOMATIC_VARIANT_SWAROOP
+               if (!MonitorChecker::instance()->ok()) {
+                       error_dialog (this, _("The required display devices are not connected correctly."));
+                       return false;
+               }
+               if (!LockFileChecker::instance()->ok()) {
+                       error_dialog (this, _("The lock file is not present."));
+                       return false;
+               }
+#endif
                if (!_film || !Config::instance()->respect_kdm_validity_periods()) {
                        return true;
                }
@@ -239,26 +319,42 @@ public:
                        return;
                }
 
-               shared_ptr<DCPContent> dcp = boost::dynamic_pointer_cast<DCPContent>(_film->content().front());
-               DCPOMATIC_ASSERT (dcp);
-               DCPExaminer ex (dcp);
-               shared_ptr<dcp::CPL> playing_cpl;
-               BOOST_FOREACH (shared_ptr<dcp::CPL> i, ex.cpls()) {
-                       if (!dcp->cpl() || i->id() == *dcp->cpl()) {
-                               playing_cpl = i;
+               FILE* f = fopen_boost(*log, "a");
+
+               /* XXX: this only logs the first piece of content; probably should be each piece? */
+
+               shared_ptr<DCPContent> dcp = dynamic_pointer_cast<DCPContent>(_film->content().front());
+               if (dcp) {
+                       DCPExaminer ex (dcp);
+                       shared_ptr<dcp::CPL> playing_cpl;
+                       BOOST_FOREACH (shared_ptr<dcp::CPL> i, ex.cpls()) {
+                               if (!dcp->cpl() || i->id() == *dcp->cpl()) {
+                                       playing_cpl = i;
+                               }
                        }
+                       DCPOMATIC_ASSERT (playing_cpl);
+
+                       fprintf (
+                               f,
+                               "%s playback-started %s %s %s\n",
+                               dcp::LocalTime().as_string().c_str(),
+                               time.timecode(_film->video_frame_rate()).c_str(),
+                               dcp->directories().front().string().c_str(),
+                               playing_cpl->annotation_text().c_str()
+                               );
+               }
+
+               shared_ptr<FFmpegContent> ffmpeg = dynamic_pointer_cast<FFmpegContent>(_film->content().front());
+               if (ffmpeg) {
+                       fprintf (
+                               f,
+                               "%s playback-started %s %s\n",
+                               dcp::LocalTime().as_string().c_str(),
+                               time.timecode(_film->video_frame_rate()).c_str(),
+                               ffmpeg->path(0).string().c_str()
+                               );
                }
-               DCPOMATIC_ASSERT (playing_cpl)
 
-               FILE* f = fopen_boost(*log, "a");
-               fprintf (
-                       f,
-                       "%s playback-started %s %s %s\n",
-                       dcp::LocalTime().as_string().c_str(),
-                       time.timecode(_film->video_frame_rate()).c_str(),
-                       dcp->directories().front().string().c_str(),
-                       playing_cpl->annotation_text().c_str()
-                       );
                fclose (f);
        }
 
@@ -276,6 +372,14 @@ public:
 
        void playback_stopped (DCPTime time)
        {
+#ifdef DCPOMATIC_VARIANT_SWAROOP
+               try {
+                       boost::filesystem::remove (Config::path("position"));
+               } catch (...) {
+                       /* Never mind */
+               }
+#endif
+
                optional<boost::filesystem::path> log = Config::instance()->player_log_file();
                if (!log) {
                        return;
@@ -293,88 +397,127 @@ public:
                Config::instance()->set_decode_reduction (reduction);
        }
 
-       void eject_dcp ()
+       void load_dcp (boost::filesystem::path dir)
        {
-               _film.reset (new Film (optional<boost::filesystem::path>()));
-               _viewer->set_film (_film);
-               _info->triggered_update ();
+               DCPOMATIC_ASSERT (_film);
+
+               reset_film ();
+               try {
+                       shared_ptr<DCPContent> dcp (new DCPContent(dir));
+                       _film->examine_and_add_content (dcp);
+                       bool const ok = display_progress (_("DCP-o-matic Player"), _("Loading content"));
+                       if (!ok || !report_errors_from_last_job(this)) {
+                               return;
+                       }
+                       Config::instance()->add_to_player_history (dir);
+               } catch (dcp::DCPReadError& e) {
+                       error_dialog (this, wxString::Format(_("Could not load a DCP from %s"), std_to_wx(dir.string())), std_to_wx(e.what()));
+               }
        }
 
-       void load_dcp (boost::filesystem::path dir)
+#ifdef DCPOMATIC_VARIANT_SWAROOP
+       optional<dcp::EncryptedKDM> get_kdm_from_url (shared_ptr<DCPContent> dcp)
        {
-               _film.reset (new Film (optional<boost::filesystem::path>()));
-               shared_ptr<DCPContent> dcp;
-               try {
-                       dcp.reset (new DCPContent (_film, dir));
-               } catch (boost::filesystem::filesystem_error& e) {
-                       error_dialog (this, _("Could not load DCP"), std_to_wx (e.what()));
-                       return;
+               ScopedTemporary temp;
+               string url = Config::instance()->kdm_server_url();
+               boost::algorithm::replace_all (url, "{CPL}", *dcp->cpl());
+               optional<dcp::EncryptedKDM> kdm;
+               if (dcp->cpl() && !get_from_url(url, false, temp)) {
+                       try {
+                               kdm = dcp::EncryptedKDM (dcp::file_to_string(temp.file()));
+                               if (kdm->cpl_id() != dcp->cpl()) {
+                                       kdm = boost::none;
+                               }
+                       } catch (std::exception& e) {
+                               /* Hey well */
+                       }
+               }
+               return kdm;
+       }
+#endif
+
+       optional<dcp::EncryptedKDM> get_kdm_from_directory (shared_ptr<DCPContent> dcp)
+       {
+               using namespace boost::filesystem;
+               optional<path> kdm_dir = Config::instance()->player_kdm_directory();
+               if (!kdm_dir) {
+                       return optional<dcp::EncryptedKDM>();
+               }
+               for (directory_iterator i = directory_iterator(*kdm_dir); i != directory_iterator(); ++i) {
+                       try {
+                               if (file_size(i->path()) < MAX_KDM_SIZE) {
+                                       dcp::EncryptedKDM kdm (dcp::file_to_string(i->path()));
+                                       if (kdm.cpl_id() == dcp->cpl()) {
+                                               return kdm;
+                                       }
+                               }
+                       } catch (std::exception& e) {
+                               /* Hey well */
+                       }
                }
+               return optional<dcp::EncryptedKDM>();
+       }
+
+       void reset_film (shared_ptr<Film> film = shared_ptr<Film>(new Film(optional<boost::filesystem::path>())))
+       {
+               _film = film;
+               _viewer->set_film (_film);
+               _film->Change.connect (bind(&DOMFrame::film_changed, this, _1, _2));
+       }
 
-               _film->examine_and_add_content (dcp, true);
-               bool const ok = progress (_("Loading DCP"));
-               if (!ok || !report_errors_from_last_job()) {
+       void film_changed (ChangeType type, Film::Property property)
+       {
+               if (type != CHANGE_TYPE_DONE || property != Film::CONTENT) {
                        return;
                }
 
-               /* The DCP has been examined and loaded */
+               _film->write_metadata (Config::path("spl.xml"));
 
-               optional<boost::filesystem::path> kdm_dir = Config::instance()->player_kdm_directory();
-               if (dcp->needs_kdm() && kdm_dir) {
-                       /* Look for a KDM */
+               if (_viewer->playing ()) {
+                       _viewer->stop ();
+               }
 
-                       optional<dcp::EncryptedKDM> kdm;
+               /* Start off as Flat */
+               _film->set_container (Ratio::from_id("185"));
 
+               BOOST_FOREACH (shared_ptr<Content> i, _film->content()) {
+                       shared_ptr<DCPContent> dcp = dynamic_pointer_cast<DCPContent>(i);
+                       if (dcp && dcp->needs_kdm()) {
+                               optional<dcp::EncryptedKDM> kdm;
 #ifdef DCPOMATIC_VARIANT_SWAROOP
-                       ScopedTemporary temp;
-                       string url = Config::instance()->kdm_server_url();
-                       boost::algorithm::replace_all (url, "{CPL}", "%1");
-                       if (dcp->cpl() && !get_from_url(String::compose(url, *dcp->cpl()), false, temp)) {
-                               try {
-                                       kdm = dcp::EncryptedKDM (dcp::file_to_string(temp.file()));
-                                       if (kdm->cpl_id() != dcp->cpl()) {
-                                               kdm = boost::none;
-                                       }
-                               } catch (std::exception& e) {
-                                       /* Hey well */
-                               }
-                       }
+                               kdm = get_kdm_from_url (dcp);
 #endif
+                               if (!kdm) {
+                                       kdm = get_kdm_from_directory (dcp);
+                               }
 
-                       if (!kdm) {
-                               using namespace boost::filesystem;
-                               for (directory_iterator i = directory_iterator(*kdm_dir); i != directory_iterator(); ++i) {
-                                       if (file_size(i->path()) < MAX_KDM_SIZE) {
-                                               try {
-                                                       kdm = dcp::EncryptedKDM(dcp::file_to_string(i->path()));
-                                                       if (kdm->cpl_id() == dcp->cpl()) {
-                                                               break;
-                                                       }
-                                               } catch (std::exception& e) {
-                                                       /* Hey well */
-                                               }
-                                       }
+                               if (kdm) {
+                                       dcp->add_kdm (*kdm);
+                                       dcp->examine (_film, shared_ptr<Job>());
                                }
                        }
 
-                       if (kdm) {
-                               dcp->add_kdm (*kdm);
-                               dcp->examine (shared_ptr<Job>());
+                       BOOST_FOREACH (shared_ptr<TextContent> j, i->text) {
+                               j->set_use (true);
                        }
-               }
 
-               setup_from_dcp (dcp);
+                       if (i->video) {
+                               Ratio const * r = Ratio::nearest_from_ratio(i->video->size().ratio());
+                               if (r->id() == "239") {
+                                       /* Any scope content means we use scope */
+                                       _film->set_container(r);
+                               }
+                       }
 
-               if (dcp->three_d()) {
-                       _film->set_three_d (true);
+                       /* Any 3D content means we use 3D mode */
+                       if (i->video && i->video->frame_type() != VIDEO_FRAME_TYPE_2D) {
+                               _film->set_three_d (true);
+                       }
                }
 
-               _viewer->set_film (_film);
                _viewer->seek (DCPTime(), true);
                _info->triggered_update ();
 
-               Config::instance()->add_to_player_history (dir);
-
                set_menu_sensitivity ();
 
                wxMenuItemList old = _cpl_menu->GetMenuItems();
@@ -382,18 +525,22 @@ public:
                        _cpl_menu->Remove (*i);
                }
 
-               DCPExaminer ex (dcp);
-               int id = ID_view_cpl;
-               BOOST_FOREACH (shared_ptr<dcp::CPL> i, ex.cpls()) {
-                       wxMenuItem* j = _cpl_menu->AppendRadioItem(
-                               id,
-                               wxString::Format("%s (%s)", std_to_wx(i->annotation_text()).data(), std_to_wx(i->id()).data())
-                               );
-                       j->Check(!dcp->cpl() || i->id() == *dcp->cpl());
-                       ++id;
+               if (_film->content().size() == 1) {
+                       /* Offer a CPL menu */
+                       shared_ptr<DCPContent> first = dynamic_pointer_cast<DCPContent>(_film->content().front());
+                       if (first) {
+                               DCPExaminer ex (first);
+                               int id = ID_view_cpl;
+                               BOOST_FOREACH (shared_ptr<dcp::CPL> i, ex.cpls()) {
+                                       wxMenuItem* j = _cpl_menu->AppendRadioItem(
+                                               id,
+                                               wxString::Format("%s (%s)", std_to_wx(i->annotation_text()).data(), std_to_wx(i->id()).data())
+                                               );
+                                       j->Check(!first->cpl() || i->id() == *first->cpl());
+                                       ++id;
+                               }
+                       }
                }
-
-               _controls->log (wxString::Format(_("Load DCP %s"), dir.filename().string().c_str()));
        }
 
 private:
@@ -418,12 +565,14 @@ private:
 #endif
 
 #ifdef __WXOSX__
-               _file_menu->Append (wxID_PREFERENCES, _("&Preferences...\tCtrl-P"));
+               wxMenuItem* prefs = _file_menu->Append (wxID_PREFERENCES, _("&Preferences...\tCtrl-P"));
 #else
                wxMenu* edit = new wxMenu;
-               edit->Append (wxID_PREFERENCES, _("&Preferences...\tCtrl-P"));
+               wxMenuItem* prefs = edit->Append (wxID_PREFERENCES, _("&Preferences...\tCtrl-P"));
 #endif
 
+               prefs->Enable (Config::instance()->have_write_permission());
+
                _cpl_menu = new wxMenu;
 
                wxMenu* view = new wxMenu;
@@ -518,11 +667,19 @@ private:
                        DCPOMATIC_ASSERT (dcp);
                        dcp->add_ov (wx_to_std(c->GetPath()));
                        JobManager::instance()->add(shared_ptr<Job>(new ExamineContentJob (_film, dcp)));
-                       bool const ok = progress (_("Loading DCP"));
-                       if (!ok || !report_errors_from_last_job()) {
+                       bool const ok = display_progress (_("DCP-o-matic Player"), _("Loading content"));
+                       if (!ok || !report_errors_from_last_job(this)) {
                                return;
                        }
-                       setup_from_dcp (dcp);
+                       BOOST_FOREACH (shared_ptr<TextContent> i, dcp->text) {
+                               i->set_use (true);
+                       }
+                       if (dcp->video) {
+                               Ratio const * r = Ratio::nearest_from_ratio(dcp->video->size().ratio());
+                               if (r) {
+                                       _film->set_container(r);
+                               }
+                       }
                }
 
                c->Destroy ();
@@ -539,7 +696,7 @@ private:
                        DCPOMATIC_ASSERT (dcp);
                        try {
                                dcp->add_kdm (dcp::EncryptedKDM (dcp::file_to_string (wx_to_std (d->GetPath ()), MAX_KDM_SIZE)));
-                               dcp->examine (shared_ptr<Job>());
+                               dcp->examine (_film, shared_ptr<Job>());
                        } catch (exception& e) {
                                error_dialog (this, wxString::Format (_("Could not load KDM.")), std_to_wx(e.what()));
                                d->Destroy ();
@@ -562,8 +719,7 @@ private:
 
        void file_close ()
        {
-               _viewer->set_film (shared_ptr<Film>());
-               _film.reset ();
+               reset_film ();
                _info->triggered_update ();
                set_menu_sensitivity ();
        }
@@ -575,6 +731,10 @@ private:
 
        void edit_preferences ()
        {
+               if (!Config::instance()->have_write_permission()) {
+                       return;
+               }
+
                if (!_config_dialog) {
                        _config_dialog = create_player_config_dialog ();
                }
@@ -597,7 +757,7 @@ private:
                }
 
                dcp->set_cpl ((*i)->id());
-               dcp->examine (shared_ptr<Job>());
+               dcp->examine (_film, shared_ptr<Job>());
        }
 
        void view_full_screen ()
@@ -639,6 +799,7 @@ private:
                _info->Show (_mode != Config::PLAYER_MODE_FULL);
                _overall_panel->SetBackgroundColour (_mode == Config::PLAYER_MODE_FULL ? wxColour(0, 0, 0) : wxNullColour);
                ShowFullScreen (_mode == Config::PLAYER_MODE_FULL);
+               _viewer->set_pad_black (_mode != Config::PLAYER_MODE_WINDOW);
 
                if (_mode == Config::PLAYER_MODE_DUAL) {
                        _dual_screen = new wxFrame (this, wxID_ANY, wxT(""));
@@ -667,7 +828,7 @@ private:
                        }
                }
 
-               setup_main_sizer (_mode != Config::PLAYER_MODE_DUAL);
+               setup_main_sizer (_mode);
        }
 
        void view_closed_captions ()
@@ -682,7 +843,7 @@ private:
 
                JobManager* jm = JobManager::instance ();
                jm->add (shared_ptr<Job> (new VerifyDCPJob (dcp->directories())));
-               bool const ok = progress (_("Verifying DCP"));
+               bool const ok = display_progress (_("DCP-o-matic Player"), _("Verifying DCP"));
                if (!ok) {
                        return;
                }
@@ -746,21 +907,33 @@ private:
                _update_news_requested = false;
        }
 
-       void config_changed ()
+       void config_changed (Config::Property prop)
        {
                /* Instantly save any config changes when using the player GUI */
                try {
                        Config::instance()->write_config();
+               } catch (FileError& e) {
+                       if (prop != Config::HISTORY) {
+                               error_dialog (
+                                       this,
+                                       wxString::Format(
+                                               _("Could not write to config file at %s.  Your changes have not been saved."),
+                                               std_to_wx(e.file().string())
+                                               )
+                                       );
+                       }
                } catch (exception& e) {
                        error_dialog (
                                this,
-                               wxString::Format (
-                                       _("Could not write to config file at %s.  Your changes have not been saved."),
-                                       std_to_wx (Config::instance()->cinemas_file().string()).data()
-                                       )
+                               _("Could not write to config file.  Your changes have not been saved.")
                                );
                }
 
+               update_from_config ();
+       }
+
+       void update_from_config ()
+       {
                for (int i = 0; i < _history_items; ++i) {
                        delete _file_menu->Remove (ID_file_history + i);
                }
@@ -821,60 +994,6 @@ private:
 
 private:
 
-       /** @return false if the task was cancelled */
-       bool progress (wxString task)
-       {
-               JobManager* jm = JobManager::instance ();
-
-               wxProgressDialog* progress = new wxProgressDialog (_("DCP-o-matic Player"), task, 100, 0, wxPD_CAN_ABORT);
-
-               bool ok = true;
-
-               while (jm->work_to_do() || signal_manager->ui_idle()) {
-                       dcpomatic_sleep (1);
-                       if (!progress->Pulse()) {
-                               /* user pressed cancel */
-                               BOOST_FOREACH (shared_ptr<Job> i, jm->get()) {
-                                       i->cancel();
-                               }
-                               ok = false;
-                               break;
-                       }
-               }
-
-               progress->Destroy ();
-               return ok;
-       }
-
-       bool report_errors_from_last_job ()
-       {
-               JobManager* jm = JobManager::instance ();
-
-               DCPOMATIC_ASSERT (!jm->get().empty());
-
-               shared_ptr<Job> last = jm->get().back();
-               if (last->finished_in_error()) {
-                       error_dialog(this, std_to_wx(last->error_summary()) + ".\n", std_to_wx(last->error_details()));
-                       return false;
-               }
-
-               return true;
-       }
-
-       void setup_from_dcp (shared_ptr<DCPContent> dcp)
-       {
-               BOOST_FOREACH (shared_ptr<TextContent> i, dcp->text) {
-                       i->set_use (true);
-               }
-
-               if (dcp->video) {
-                       Ratio const * r = Ratio::nearest_from_ratio(dcp->video->size().ratio());
-                       if (r) {
-                               _film->set_container(r);
-                       }
-               }
-       }
-
        wxFrame* _dual_screen;
        bool _update_news_requested;
        PlayerInformation* _info;
@@ -988,6 +1107,8 @@ private:
                */
                Config::drop ();
 
+               signal_manager = new wxSignalManager (this);
+
                _frame = new DOMFrame ();
                SetTopWindow (_frame);
                _frame->Maximize ();
@@ -996,8 +1117,6 @@ private:
                }
                _frame->Show ();
 
-               signal_manager = new wxSignalManager (this);
-
                PlayServer* server = new PlayServer (_frame);
                new thread (boost::bind (&PlayServer::run, server));