X-Git-Url: https://main.carlh.net/gitweb/?p=dcpomatic.git;a=blobdiff_plain;f=src%2Ftools%2Fdcpomatic.cc;h=01aa0158b5b7596f5cc96f1008e0caad91ce6621;hp=23c1e500512a750f17468445a2cfcfd1cd78f7de;hb=391d85619ac19a2a93696ddc35c222eb9bb5d9d6;hpb=8102046b2f29e0c7b234c29bf204b056cb30e64f diff --git a/src/tools/dcpomatic.cc b/src/tools/dcpomatic.cc index 23c1e5005..01aa0158b 100644 --- a/src/tools/dcpomatic.cc +++ b/src/tools/dcpomatic.cc @@ -30,7 +30,7 @@ #include #include #include -#include +#include #include "wx/film_viewer.h" #include "wx/film_editor.h" #include "wx/job_manager_view.h" @@ -45,6 +45,7 @@ #include "wx/servers_list_dialog.h" #include "wx/hints_dialog.h" #include "wx/update_dialog.h" +#include "wx/content_panel.h" #include "lib/film.h" #include "lib/config.h" #include "lib/util.h" @@ -63,8 +64,8 @@ using std::cout; using std::string; +using std::vector; using std::wstring; -using std::stringstream; using std::map; using std::make_pair; using std::list; @@ -72,26 +73,14 @@ using std::exception; using boost::shared_ptr; using boost::dynamic_pointer_cast; -static FilmEditor* film_editor = 0; -static FilmViewer* film_viewer = 0; -static shared_ptr film; -static std::string film_to_load; -static std::string film_to_create; -static std::string content_to_add; -static wxMenu* jobs_menu = 0; - -static void set_menu_sensitivity (); - -// #define DCPOMATIC_WINDOWS_CONSOLE 1 - class FilmChangedDialog { public: - FilmChangedDialog () + FilmChangedDialog (string name) { _dialog = new wxMessageDialog ( 0, - wxString::Format (_("Save changes to film \"%s\" before closing?"), std_to_wx (film->name ()).data()), + wxString::Format (_("Save changes to film \"%s\" before closing?"), std_to_wx (name).data()), _("Film changed"), wxYES_NO | wxYES_DEFAULT | wxICON_QUESTION ); @@ -114,165 +103,34 @@ private: wxMessageDialog* _dialog; }; - -static void -maybe_save_then_delete_film () -{ - if (!film) { - return; - } - - if (film->dirty ()) { - FilmChangedDialog d; - switch (d.run ()) { - case wxID_NO: - break; - case wxID_YES: - film->write_metadata (); - break; - } - } - - film.reset (); -} - -static void -check_film_state_version (int v) -{ - if (v == 4) { - error_dialog ( - 0, - _("This film was created with an old version of DVD-o-matic and may not load correctly " - "in this version. Please check the film's settings carefully.") - ); - } -} - -static void -load_film (boost::filesystem::path file) -{ - film.reset (new Film (file)); - list const notes = film->read_metadata (); - check_film_state_version (film->state_version ()); - for (list::const_iterator i = notes.begin(); i != notes.end(); ++i) { - error_dialog (0, std_to_wx (*i)); - } -} - -#define ALWAYS 0x0 -#define NEEDS_FILM 0x1 -#define NOT_DURING_DCP_CREATION 0x2 -#define NEEDS_CPL 0x4 +#define ALWAYS 0x0 +#define NEEDS_FILM 0x1 +#define NOT_DURING_DCP_CREATION 0x2 +#define NEEDS_CPL 0x4 +#define NEEDS_SELECTED_VIDEO_CONTENT 0x8 map menu_items; -static void -add_item (wxMenu* menu, wxString text, int id, int sens) -{ - wxMenuItem* item = menu->Append (id, text); - menu_items.insert (make_pair (item, sens)); -} - -static void -set_menu_sensitivity () -{ - list > jobs = JobManager::instance()->get (); - list >::iterator i = jobs.begin(); - while (i != jobs.end() && dynamic_pointer_cast (*i) == 0) { - ++i; - } - bool const dcp_creation = (i != jobs.end ()) && !(*i)->finished (); - bool const have_cpl = film && !film->cpls().empty (); - - for (map::iterator j = menu_items.begin(); j != menu_items.end(); ++j) { - - bool enabled = true; - - if ((j->second & NEEDS_FILM) && film == 0) { - enabled = false; - } - - if ((j->second & NOT_DURING_DCP_CREATION) && dcp_creation) { - enabled = false; - } - - if ((j->second & NEEDS_CPL) && !have_cpl) { - enabled = false; - } - - j->first->Enable (enabled); - } -} - enum { ID_file_new = 1, ID_file_open, ID_file_save, ID_file_properties, + ID_file_history, + /* Allow spare IDs after _history for the recent files list */ + ID_content_scale_to_fit_width = 100, + ID_content_scale_to_fit_height, ID_jobs_make_dcp, ID_jobs_make_kdms, ID_jobs_send_dcp_to_tms, ID_jobs_show_dcp, ID_tools_hints, ID_tools_encoding_servers, - ID_tools_check_for_updates + ID_tools_check_for_updates, + /* IDs for shortcuts (with no associated menu item) */ + ID_add_file }; -static void -setup_menu (wxMenuBar* m) -{ - wxMenu* file = new wxMenu; - add_item (file, _("New..."), ID_file_new, ALWAYS); - add_item (file, _("&Open..."), ID_file_open, ALWAYS); - file->AppendSeparator (); - add_item (file, _("&Save"), ID_file_save, NEEDS_FILM); - file->AppendSeparator (); - add_item (file, _("&Properties..."), ID_file_properties, NEEDS_FILM); -#ifndef __WXOSX__ - file->AppendSeparator (); -#endif - -#ifdef __WXOSX__ - add_item (file, _("&Exit"), wxID_EXIT, ALWAYS); -#else - add_item (file, _("&Quit"), wxID_EXIT, ALWAYS); -#endif - - -#ifdef __WXOSX__ - add_item (file, _("&Preferences..."), wxID_PREFERENCES, ALWAYS); -#else - wxMenu* edit = new wxMenu; - add_item (edit, _("&Preferences..."), wxID_PREFERENCES, ALWAYS); -#endif - - jobs_menu = new wxMenu; - add_item (jobs_menu, _("&Make DCP"), ID_jobs_make_dcp, NEEDS_FILM | NOT_DURING_DCP_CREATION); - add_item (jobs_menu, _("Make &KDMs..."), ID_jobs_make_kdms, NEEDS_FILM); - add_item (jobs_menu, _("&Send DCP to TMS"), ID_jobs_send_dcp_to_tms, NEEDS_FILM | NOT_DURING_DCP_CREATION | NEEDS_CPL); - add_item (jobs_menu, _("S&how DCP"), ID_jobs_show_dcp, NEEDS_FILM | NOT_DURING_DCP_CREATION | NEEDS_CPL); - - wxMenu* tools = new wxMenu; - add_item (tools, _("Hints..."), ID_tools_hints, 0); - add_item (tools, _("Encoding servers..."), ID_tools_encoding_servers, 0); - add_item (tools, _("Check for updates"), ID_tools_check_for_updates, 0); - - wxMenu* help = new wxMenu; -#ifdef __WXOSX__ - add_item (help, _("About DCP-o-matic"), wxID_ABOUT, ALWAYS); -#else - add_item (help, _("About"), wxID_ABOUT, ALWAYS); -#endif - - m->Append (file, _("&File")); -#ifndef __WXOSX__ - m->Append (edit, _("&Edit")); -#endif - m->Append (jobs_menu, _("&Jobs")); - m->Append (tools, _("&Tools")); - m->Append (help, _("&Help")); -} - class Frame : public wxFrame { public: @@ -281,33 +139,47 @@ public: , _hints_dialog (0) , _servers_list_dialog (0) , _config_dialog (0) + , _file_menu (0) + , _history_items (0) + , _history_position (0) + , _history_separator (0) { -#if defined(DCPOMATIC_WINDOWS) && defined(DCPOMATIC_WINDOWS_CONSOLE) - AllocConsole(); - - HANDLE handle_out = GetStdHandle(STD_OUTPUT_HANDLE); - int hCrt = _open_osfhandle((intptr_t) handle_out, _O_TEXT); - FILE* hf_out = _fdopen(hCrt, "w"); - setvbuf(hf_out, NULL, _IONBF, 1); - *stdout = *hf_out; - - HANDLE handle_in = GetStdHandle(STD_INPUT_HANDLE); - hCrt = _open_osfhandle((intptr_t) handle_in, _O_TEXT); - FILE* hf_in = _fdopen(hCrt, "r"); - setvbuf(hf_in, NULL, _IONBF, 128); - *stdin = *hf_in; +#if defined(DCPOMATIC_WINDOWS) + if (Config::instance()->win32_console ()) { + AllocConsole(); + + HANDLE handle_out = GetStdHandle(STD_OUTPUT_HANDLE); + int hCrt = _open_osfhandle((intptr_t) handle_out, _O_TEXT); + FILE* hf_out = _fdopen(hCrt, "w"); + setvbuf(hf_out, NULL, _IONBF, 1); + *stdout = *hf_out; + + HANDLE handle_in = GetStdHandle(STD_INPUT_HANDLE); + hCrt = _open_osfhandle((intptr_t) handle_in, _O_TEXT); + FILE* hf_in = _fdopen(hCrt, "r"); + setvbuf(hf_in, NULL, _IONBF, 128); + *stdin = *hf_in; + + cout << "DCP-o-matic is starting." << "\n"; + } #endif wxMenuBar* bar = new wxMenuBar; setup_menu (bar); SetMenuBar (bar); + _config_changed_connection = Config::instance()->Changed.connect (boost::bind (&Frame::config_changed, this)); + config_changed (); + Bind (wxEVT_COMMAND_MENU_SELECTED, boost::bind (&Frame::file_new, this), ID_file_new); Bind (wxEVT_COMMAND_MENU_SELECTED, boost::bind (&Frame::file_open, this), ID_file_open); Bind (wxEVT_COMMAND_MENU_SELECTED, boost::bind (&Frame::file_save, this), ID_file_save); Bind (wxEVT_COMMAND_MENU_SELECTED, boost::bind (&Frame::file_properties, this), ID_file_properties); + Bind (wxEVT_COMMAND_MENU_SELECTED, boost::bind (&Frame::file_history, this, _1), ID_file_history, ID_file_history + HISTORY_SIZE); Bind (wxEVT_COMMAND_MENU_SELECTED, boost::bind (&Frame::file_exit, this), wxID_EXIT); Bind (wxEVT_COMMAND_MENU_SELECTED, boost::bind (&Frame::edit_preferences, this), wxID_PREFERENCES); + Bind (wxEVT_COMMAND_MENU_SELECTED, boost::bind (&Frame::content_scale_to_fit_width, this), ID_content_scale_to_fit_width); + Bind (wxEVT_COMMAND_MENU_SELECTED, boost::bind (&Frame::content_scale_to_fit_height, this), ID_content_scale_to_fit_height); Bind (wxEVT_COMMAND_MENU_SELECTED, boost::bind (&Frame::jobs_make_dcp, this), ID_jobs_make_dcp); Bind (wxEVT_COMMAND_MENU_SELECTED, boost::bind (&Frame::jobs_make_kdms, this), ID_jobs_make_kdms); Bind (wxEVT_COMMAND_MENU_SELECTED, boost::bind (&Frame::jobs_send_dcp_to_tms, this), ID_jobs_send_dcp_to_tms); @@ -319,56 +191,98 @@ public: Bind (wxEVT_CLOSE_WINDOW, boost::bind (&Frame::close, this, _1)); + wxAcceleratorEntry accel[1]; + accel[0].Set (wxACCEL_CTRL, static_cast('A'), ID_add_file); + Bind (wxEVT_MENU, boost::bind (&ContentPanel::add_file_clicked, _film_editor->content_panel()), ID_add_file); + wxAcceleratorTable accel_table (1, accel); + SetAcceleratorTable (accel_table); + /* Use a panel as the only child of the Frame so that we avoid the dark-grey background on Windows. */ wxPanel* overall_panel = new wxPanel (this, wxID_ANY); - film_editor = new FilmEditor (film, overall_panel); - film_viewer = new FilmViewer (film, overall_panel); + _film_editor = new FilmEditor (overall_panel); + _film_viewer = new FilmViewer (overall_panel); JobManagerView* job_manager_view = new JobManagerView (overall_panel, static_cast (0)); wxBoxSizer* right_sizer = new wxBoxSizer (wxVERTICAL); - right_sizer->Add (film_viewer, 2, wxEXPAND | wxALL, 6); + right_sizer->Add (_film_viewer, 2, wxEXPAND | wxALL, 6); right_sizer->Add (job_manager_view, 1, wxEXPAND | wxALL, 6); wxBoxSizer* main_sizer = new wxBoxSizer (wxHORIZONTAL); - main_sizer->Add (film_editor, 1, wxEXPAND | wxALL, 6); + main_sizer->Add (_film_editor, 1, wxEXPAND | wxALL, 6); main_sizer->Add (right_sizer, 2, wxEXPAND | wxALL, 6); set_menu_sensitivity (); - film_editor->FileChanged.connect (bind (&Frame::file_changed, this, _1)); - if (film) { - file_changed (film->directory ()); - } else { - file_changed (""); - } + _film_editor->FileChanged.connect (bind (&Frame::file_changed, this, _1)); + file_changed (""); - JobManager::instance()->ActiveJobsChanged.connect (boost::bind (set_menu_sensitivity)); + JobManager::instance()->ActiveJobsChanged.connect (boost::bind (&Frame::set_menu_sensitivity, this)); - set_film (); overall_panel->SetSizer (main_sizer); } -private: + void new_film (boost::filesystem::path path) + { + shared_ptr film (new Film (path)); + film->write_metadata (); + film->set_name (path.filename().generic_string()); + set_film (film); + } - void set_film () + void load_film (boost::filesystem::path file) + try { - film_viewer->set_film (film); - film_editor->set_film (film); + maybe_save_then_delete_film (); + + shared_ptr film (new Film (file)); + list const notes = film->read_metadata (); + + if (film->state_version() == 4) { + error_dialog ( + 0, + _("This film was created with an old version of DVD-o-matic and may not load correctly " + "in this version. Please check the film's settings carefully.") + ); + } + + for (list::const_iterator i = notes.begin(); i != notes.end(); ++i) { + error_dialog (0, std_to_wx (*i)); + } + + set_film (film); + } + catch (std::exception& e) { + wxString p = std_to_wx (file.string ()); + wxCharBuffer b = p.ToUTF8 (); + error_dialog (this, wxString::Format (_("Could not open film at %s (%s)"), p.data(), std_to_wx (e.what()).data())); + } + + void set_film (shared_ptr film) + { + _film = film; + _film_viewer->set_film (_film); + _film_editor->set_film (_film); set_menu_sensitivity (); + Config::instance()->add_to_history (_film->directory ()); + } + + shared_ptr film () const { + return _film; } + +private: void file_changed (boost::filesystem::path f) { - stringstream s; - s << wx_to_std (_("DCP-o-matic")); + string s = wx_to_std (_("DCP-o-matic")); if (!f.empty ()) { - s << " - " << f.string (); + s += " - " + f.string (); } - SetTitle (std_to_wx (s.str())); + SetTitle (std_to_wx (s)); } void file_new () @@ -398,10 +312,7 @@ private: } maybe_save_then_delete_film (); - film.reset (new Film (d->get_path ())); - film->write_metadata (); - film->set_name (boost::filesystem::path (d->get_path()).filename().generic_string()); - set_film (); + new_film (d->get_path ()); } d->Destroy (); @@ -417,7 +328,7 @@ private: ); int r; - while (1) { + while (true) { r = c->ShowModal (); if (r == wxID_OK && c->GetPath() == wxStandardPaths::Get().GetDocumentsDir()) { error_dialog (this, _("You did not select a folder. Make sure that you select a folder before clicking Open.")); @@ -427,15 +338,7 @@ private: } if (r == wxID_OK) { - maybe_save_then_delete_film (); - try { - load_film (wx_to_std (c->GetPath ())); - set_film (); - } catch (std::exception& e) { - wxString p = c->GetPath (); - wxCharBuffer b = p.ToUTF8 (); - error_dialog (this, wxString::Format (_("Could not open film at %s (%s)"), p.data(), std_to_wx (e.what()).data())); - } + load_film (wx_to_std (c->GetPath ())); } c->Destroy (); @@ -443,15 +346,24 @@ private: void file_save () { - film->write_metadata (); + _film->write_metadata (); } void file_properties () { - PropertiesDialog* d = new PropertiesDialog (this, film); + PropertiesDialog* d = new PropertiesDialog (this, _film); d->ShowModal (); d->Destroy (); } + + void file_history (wxCommandEvent& event) + { + vector history = Config::instance()->history (); + int n = event.GetId() - ID_file_history; + if (n >= 0 && n < static_cast (history.size ())) { + load_film (history[n]); + } + } void file_exit () { @@ -472,22 +384,22 @@ private: double required; double available; - if (!film->should_be_enough_disk_space (required, available)) { + if (!_film->should_be_enough_disk_space (required, available)) { if (!confirm_dialog (this, wxString::Format (_("The DCP for this film will take up about %.1f Gb, and the disk that you are using only has %.1f Gb available. Do you want to continue anyway?"), required, available))) { return; } } - JobWrapper::make_dcp (this, film); + JobWrapper::make_dcp (this, _film); } void jobs_make_kdms () { - if (!film) { + if (!_film) { return; } - KDMDialog* d = new KDMDialog (this, film); + KDMDialog* d = new KDMDialog (this, _film); if (d->ShowModal () != wxID_OK) { d->Destroy (); return; @@ -495,13 +407,13 @@ private: try { if (d->write_to ()) { - write_kdm_files (film, d->screens (), d->cpl (), d->from (), d->until (), d->directory ()); + write_kdm_files (_film, d->screens (), d->cpl (), d->from (), d->until (), d->formulation (), d->directory ()); } else { JobManager::instance()->add ( - shared_ptr (new SendKDMEmailJob (film, d->screens (), d->cpl (), d->from (), d->until ())) + shared_ptr (new SendKDMEmailJob (_film, d->screens (), d->cpl (), d->from (), d->until (), d->formulation ())) ); } - } catch (libdcp::NotEncryptedError& e) { + } catch (dcp::NotEncryptedError& e) { error_dialog (this, _("CPL's content is not encrypted.")); } catch (exception& e) { error_dialog (this, e.what ()); @@ -511,30 +423,46 @@ private: d->Destroy (); } + + void content_scale_to_fit_width () + { + VideoContentList vc = _film_editor->content_panel()->selected_video (); + for (VideoContentList::iterator i = vc.begin(); i != vc.end(); ++i) { + (*i)->scale_and_crop_to_fit_width (); + } + } + + void content_scale_to_fit_height () + { + VideoContentList vc = _film_editor->content_panel()->selected_video (); + for (VideoContentList::iterator i = vc.begin(); i != vc.end(); ++i) { + (*i)->scale_and_crop_to_fit_height (); + } + } void jobs_send_dcp_to_tms () { - film->send_dcp_to_tms (); + _film->send_dcp_to_tms (); } void jobs_show_dcp () { #ifdef __WXMSW__ - string d = film->directory().string (); + string d = _film->directory().string (); wstring w; w.assign (d.begin(), d.end()); ShellExecute (0, L"open", w.c_str(), 0, 0, SW_SHOWDEFAULT); #else int r = system ("which nautilus"); if (WEXITSTATUS (r) == 0) { - r = system (string ("nautilus " + film->directory().string()).c_str ()); + r = system (string ("nautilus " + _film->directory().string()).c_str ()); if (WEXITSTATUS (r)) { error_dialog (this, _("Could not show DCP (could not run nautilus)")); } } else { int r = system ("which konqueror"); if (WEXITSTATUS (r) == 0) { - r = system (string ("konqueror " + film->directory().string()).c_str ()); + r = system (string ("konqueror " + _film->directory().string()).c_str ()); if (WEXITSTATUS (r)) { error_dialog (this, _("Could not show DCP (could not run konqueror)")); } @@ -546,7 +474,7 @@ private: void tools_hints () { if (!_hints_dialog) { - _hints_dialog = new HintsDialog (this, film); + _hints_dialog = new HintsDialog (this, _film); } _hints_dialog->Show (); @@ -598,14 +526,181 @@ private: return; } + /* We don't want to hear about any more configuration changes, since they + cause the File menu to be altered, which itself will be deleted around + now (without, as far as I can see, any way for us to find out). + */ + _config_changed_connection.disconnect (); + maybe_save_then_delete_film (); - ev.Skip (); } + void set_menu_sensitivity () + { + list > jobs = JobManager::instance()->get (); + list >::iterator i = jobs.begin(); + while (i != jobs.end() && dynamic_pointer_cast (*i) == 0) { + ++i; + } + bool const dcp_creation = (i != jobs.end ()) && !(*i)->finished (); + bool const have_cpl = _film && !_film->cpls().empty (); + bool const have_selected_video_content = !_film_editor->content_panel()->selected_video().empty(); + + for (map::iterator j = menu_items.begin(); j != menu_items.end(); ++j) { + + bool enabled = true; + + if ((j->second & NEEDS_FILM) && !_film) { + enabled = false; + } + + if ((j->second & NOT_DURING_DCP_CREATION) && dcp_creation) { + enabled = false; + } + + if ((j->second & NEEDS_CPL) && !have_cpl) { + enabled = false; + } + + if ((j->second & NEEDS_SELECTED_VIDEO_CONTENT) && !have_selected_video_content) { + enabled = false; + } + + j->first->Enable (enabled); + } + } + + void maybe_save_then_delete_film () + { + if (!_film) { + return; + } + + if (_film->dirty ()) { + FilmChangedDialog d (_film->name ()); + switch (d.run ()) { + case wxID_NO: + break; + case wxID_YES: + _film->write_metadata (); + break; + } + } + + _film.reset (); + } + + void add_item (wxMenu* menu, wxString text, int id, int sens) + { + wxMenuItem* item = menu->Append (id, text); + menu_items.insert (make_pair (item, sens)); + } + + void setup_menu (wxMenuBar* m) + { + _file_menu = new wxMenu; + add_item (_file_menu, _("New...\tCtrl-N"), ID_file_new, ALWAYS); + add_item (_file_menu, _("&Open...\tCtrl-O"), ID_file_open, ALWAYS); + _file_menu->AppendSeparator (); + add_item (_file_menu, _("&Save\tCtrl-S"), ID_file_save, NEEDS_FILM); + _file_menu->AppendSeparator (); + add_item (_file_menu, _("&Properties..."), ID_file_properties, NEEDS_FILM); + + _history_position = _file_menu->GetMenuItems().GetCount(); + +#ifndef __WXOSX__ + _file_menu->AppendSeparator (); +#endif + +#ifdef __WXOSX__ + add_item (_file_menu, _("&Exit"), wxID_EXIT, ALWAYS); +#else + add_item (_file_menu, _("&Quit"), wxID_EXIT, ALWAYS); +#endif + +#ifdef __WXOSX__ + add_item (_file_menu, _("&Preferences...\tCtrl-P"), wxID_PREFERENCES, ALWAYS); +#else + wxMenu* edit = new wxMenu; + add_item (edit, _("&Preferences...\tCtrl-P"), wxID_PREFERENCES, ALWAYS); +#endif + + wxMenu* content = new wxMenu; + add_item (content, _("Scale to fit &width"), ID_content_scale_to_fit_width, NEEDS_FILM | NEEDS_SELECTED_VIDEO_CONTENT); + add_item (content, _("Scale to fit &height"), ID_content_scale_to_fit_height, NEEDS_FILM | NEEDS_SELECTED_VIDEO_CONTENT); + + wxMenu* jobs_menu = new wxMenu; + add_item (jobs_menu, _("&Make DCP\tCtrl-M"), ID_jobs_make_dcp, NEEDS_FILM | NOT_DURING_DCP_CREATION); + add_item (jobs_menu, _("Make &KDMs...\tCtrl-K"), ID_jobs_make_kdms, NEEDS_FILM); + add_item (jobs_menu, _("&Send DCP to TMS"), ID_jobs_send_dcp_to_tms, NEEDS_FILM | NOT_DURING_DCP_CREATION | NEEDS_CPL); + add_item (jobs_menu, _("S&how DCP"), ID_jobs_show_dcp, NEEDS_FILM | NOT_DURING_DCP_CREATION | NEEDS_CPL); + + wxMenu* tools = new wxMenu; + add_item (tools, _("Hints...\tCtrl-H"), ID_tools_hints, 0); + add_item (tools, _("Encoding servers..."), ID_tools_encoding_servers, 0); + add_item (tools, _("Check for updates"), ID_tools_check_for_updates, 0); + + wxMenu* help = new wxMenu; +#ifdef __WXOSX__ + add_item (help, _("About DCP-o-matic"), wxID_ABOUT, ALWAYS); +#else + add_item (help, _("About"), wxID_ABOUT, ALWAYS); +#endif + + m->Append (_file_menu, _("&File")); +#ifndef __WXOSX__ + m->Append (edit, _("&Edit")); +#endif + m->Append (content, _("&Content")); + m->Append (jobs_menu, _("&Jobs")); + m->Append (tools, _("&Tools")); + m->Append (help, _("&Help")); + } + + void config_changed () + { + for (int i = 0; i < _history_items; ++i) { + delete _file_menu->Remove (ID_file_history + i); + } + + if (_history_separator) { + _file_menu->Remove (_history_separator); + } + delete _history_separator; + _history_separator = 0; + + int pos = _history_position; + + vector history = Config::instance()->history (); + + if (!history.empty ()) { + _history_separator = _file_menu->InsertSeparator (pos++); + } + + for (size_t i = 0; i < history.size(); ++i) { + SafeStringStream s; + if (i < 9) { + s << "&" << (i + 1) << " "; + } + s << history[i].string(); + _file_menu->Insert (pos++, ID_file_history + i, std_to_wx (s.str ())); + } + + _history_items = history.size (); + } + + FilmEditor* _film_editor; + FilmViewer* _film_viewer; HintsDialog* _hints_dialog; ServersListDialog* _servers_list_dialog; wxPreferencesEditor* _config_dialog; + wxMenu* _file_menu; + shared_ptr _film; + int _history_items; + int _history_position; + wxMenuItem* _history_separator; + boost::signals2::scoped_connection _config_changed_connection; }; static const wxCmdLineEntryDesc command_line_description[] = { @@ -659,29 +754,26 @@ class App : public wxApp */ Config::drop (); - if (!film_to_load.empty() && boost::filesystem::is_directory (film_to_load)) { + _frame = new Frame (_("DCP-o-matic")); + SetTopWindow (_frame); + _frame->Maximize (); + _frame->Show (); + + if (!_film_to_load.empty() && boost::filesystem::is_directory (_film_to_load)) { try { - load_film (film_to_load); + _frame->load_film (_film_to_load); } catch (exception& e) { - error_dialog (0, std_to_wx (String::compose (wx_to_std (_("Could not load film %1 (%2)")), film_to_load, e.what()))); + error_dialog (0, std_to_wx (String::compose (wx_to_std (_("Could not load film %1 (%2)")), _film_to_load, e.what()))); } } - if (!film_to_create.empty ()) { - film.reset (new Film (film_to_create)); - film->write_metadata (); - film->set_name (boost::filesystem::path (film_to_create).filename().generic_string ()); - } - - if (!content_to_add.empty ()) { - film->examine_and_add_content (content_factory (film, content_to_add)); + if (!_film_to_create.empty ()) { + _frame->new_film (_film_to_create); + if (!_content_to_add.empty ()) { + _frame->film()->examine_and_add_content (content_factory (_frame->film(), _content_to_add)); + } } - _frame = new Frame (_("DCP-o-matic")); - SetTopWindow (_frame); - _frame->Maximize (); - _frame->Show (); - ui_signaller = new wxUISignaller (this); Bind (wxEVT_IDLE, boost::bind (&App::idle, this)); @@ -689,10 +781,6 @@ class App : public wxApp _timer.reset (new wxTimer (this)); _timer->Start (1000); - if (film) { - check_film_state_version (film->state_version ()); - } - UpdateChecker::instance()->StateChanged.connect (boost::bind (&App::update_checker_state_changed, this)); if (Config::instance()->check_for_updates ()) { UpdateChecker::instance()->run (); @@ -716,26 +804,35 @@ class App : public wxApp { if (parser.GetParamCount() > 0) { if (parser.Found (wxT ("new"))) { - film_to_create = wx_to_std (parser.GetParam (0)); + _film_to_create = wx_to_std (parser.GetParam (0)); } else { - film_to_load = wx_to_std (parser.GetParam (0)); + _film_to_load = wx_to_std (parser.GetParam (0)); } } wxString content; if (parser.Found (wxT ("content"), &content)) { - content_to_add = wx_to_std (content); + _content_to_add = wx_to_std (content); } return true; } + /* An unhandled exception has occurred inside the main event loop */ bool OnExceptionInMainLoop () { - error_dialog (0, _("An unknown exception occurred. Please report this problem to the DCP-o-matic author (carl@dcpomatic.com).")); + try { + throw; + } catch (exception& e) { + error_dialog (0, wxString::Format (_("An exception occurred (%s). Please report this problem to the DCP-o-matic author (carl@dcpomatic.com)."), e.what ())); + } catch (...) { + error_dialog (0, _("An unknown exception occurred. Please report this problem to the DCP-o-matic author (carl@dcpomatic.com).")); + } + + /* This will terminate the program */ return false; } - + void OnUnhandledException () { error_dialog (0, _("An unknown exception occurred. Please report this problem to the DCP-o-matic author (carl@dcpomatic.com).")); @@ -785,6 +882,9 @@ class App : public wxApp Frame* _frame; shared_ptr _timer; + string _film_to_load; + string _film_to_create; + string _content_to_add; }; IMPLEMENT_APP (App)