From: Carl Hetherington <cth@carlh.net>
Date: Sun, 23 Dec 2018 22:05:04 +0000 (+0000)
Subject: Splash screen patch for player.
X-Git-Tag: v2.13.91~7
X-Git-Url: https://git.carlh.net/gitweb/?p=dcpomatic.git;a=commitdiff_plain;h=ca904732e0833b8da2284bc81deef48ce67e1323

Splash screen patch for player.
---

diff --git a/src/tools/dcpomatic_player.cc b/src/tools/dcpomatic_player.cc
index f07b7c3aa..1b55df7d4 100644
--- a/src/tools/dcpomatic_player.cc
+++ b/src/tools/dcpomatic_player.cc
@@ -989,85 +989,90 @@ public:
 private:
 
 	bool OnInit ()
-	try
 	{
-		wxInitAllImageHandlers ();
+		wxSplashScreen* splash = 0;
+		try {
+			wxInitAllImageHandlers ();
 
-		Config::FailedToLoad.connect (boost::bind (&App::config_failed_to_load, this));
-		Config::Warning.connect (boost::bind (&App::config_warning, this, _1));
+			Config::FailedToLoad.connect (boost::bind (&App::config_failed_to_load, this));
+			Config::Warning.connect (boost::bind (&App::config_warning, this, _1));
 
-		wxSplashScreen* splash = maybe_show_splash ();
+			wxSplashScreen* splash = maybe_show_splash ();
 
-		SetAppName (_("DCP-o-matic Player"));
+			SetAppName (_("DCP-o-matic Player"));
 
-		if (!wxApp::OnInit()) {
-			return false;
-		}
+			if (!wxApp::OnInit()) {
+				return false;
+			}
 
 #ifdef DCPOMATIC_LINUX
-		unsetenv ("UBUNTU_MENUPROXY");
+			unsetenv ("UBUNTU_MENUPROXY");
 #endif
 
 #ifdef __WXOSX__
-		ProcessSerialNumber serial;
-		GetCurrentProcess (&serial);
-		TransformProcessType (&serial, kProcessTransformToForegroundApplication);
+			ProcessSerialNumber serial;
+			GetCurrentProcess (&serial);
+			TransformProcessType (&serial, kProcessTransformToForegroundApplication);
 #endif
 
-		dcpomatic_setup_path_encoding ();
-
-		/* Enable i18n; this will create a Config object
-		   to look for a force-configured language.  This Config
-		   object will be wrong, however, because dcpomatic_setup
-		   hasn't yet been called and there aren't any filters etc.
-		   set up yet.
-		*/
-		dcpomatic_setup_i18n ();
-
-		/* Set things up, including filters etc.
-		   which will now be internationalised correctly.
-		*/
-		dcpomatic_setup ();
-
-		/* Force the configuration to be re-loaded correctly next
-		   time it is needed.
-		*/
-		Config::drop ();
+			dcpomatic_setup_path_encoding ();
+
+			/* Enable i18n; this will create a Config object
+			   to look for a force-configured language.  This Config
+			   object will be wrong, however, because dcpomatic_setup
+			   hasn't yet been called and there aren't any filters etc.
+			   set up yet.
+			*/
+			dcpomatic_setup_i18n ();
+
+			/* Set things up, including filters etc.
+			   which will now be internationalised correctly.
+			*/
+			dcpomatic_setup ();
+
+			/* Force the configuration to be re-loaded correctly next
+			   time it is needed.
+			*/
+			Config::drop ();
+
+			signal_manager = new wxSignalManager (this);
+
+			_frame = new DOMFrame ();
+			SetTopWindow (_frame);
+			_frame->Maximize ();
+			if (splash) {
+				splash->Destroy ();
+				splash = 0;
+			}
+			_frame->Show ();
 
-		signal_manager = new wxSignalManager (this);
+			PlayServer* server = new PlayServer (_frame);
+			new thread (boost::bind (&PlayServer::run, server));
 
-		_frame = new DOMFrame ();
-		SetTopWindow (_frame);
-		_frame->Maximize ();
-		if (splash) {
-			splash->Destroy ();
-		}
-		_frame->Show ();
+			if (!_dcp_to_load.empty() && boost::filesystem::is_directory (_dcp_to_load)) {
+				try {
+					_frame->load_dcp (_dcp_to_load);
+				} catch (exception& e) {
+					error_dialog (0, std_to_wx (String::compose (wx_to_std (_("Could not load DCP %1.")), _dcp_to_load)), std_to_wx(e.what()));
+				}
+			}
 
-		PlayServer* server = new PlayServer (_frame);
-		new thread (boost::bind (&PlayServer::run, server));
+			Bind (wxEVT_IDLE, boost::bind (&App::idle, this));
 
-		if (!_dcp_to_load.empty() && boost::filesystem::is_directory (_dcp_to_load)) {
-			try {
-				_frame->load_dcp (_dcp_to_load);
-			} catch (exception& e) {
-				error_dialog (0, std_to_wx (String::compose (wx_to_std (_("Could not load DCP %1.")), _dcp_to_load)), std_to_wx(e.what()));
+			if (Config::instance()->check_for_updates ()) {
+				UpdateChecker::instance()->run ();
 			}
 		}
-
-		Bind (wxEVT_IDLE, boost::bind (&App::idle, this));
-
-		if (Config::instance()->check_for_updates ()) {
-			UpdateChecker::instance()->run ();
+		catch (exception& e)
+		{
+			if (splash) {
+				splash->Destroy ();
+			}
+			error_dialog (0, _("DCP-o-matic Player could not start."), std_to_wx(e.what()));
 		}
 
 		return true;
 	}
-	catch (exception& e)
-	{
-		error_dialog (0, _("DCP-o-matic Player could not start."), std_to_wx(e.what()));
-		return true;
-	}
 
 	void OnInitCmdLine (wxCmdLineParser& parser)
 	{