Various OSX warnings fixes.
[dcpomatic.git] / src / tools / dcpomatic.cc
index 13569d11d7acd648961358ae1f36cf18fd268ea4..75b4025aa554ef2d7956e77fd51fb612c66c3bfc 100644 (file)
@@ -98,9 +98,6 @@
 #ifdef __WXMSW__
 #include <shellapi.h>
 #endif
-#ifdef __WXOSX__
-#include <ApplicationServices/ApplicationServices.h>
-#endif
 #include <boost/filesystem.hpp>
 #include <boost/noncopyable.hpp>
 #include <boost/foreach.hpp>
@@ -812,7 +809,7 @@ private:
        }
 
        /** @return false if we succeeded, true if not */
-       bool send_to_other_tool (int port, function<void(boost::filesystem::path)> start, string message)
+       bool send_to_other_tool (int port, function<void()> start, string message)
        {
                /* i = 0; try to connect via socket
                   i = 1; try again, and then try to start the tool
@@ -838,7 +835,7 @@ private:
                        }
 
                        if (i == 1) {
-                               start (wx_to_std (wxStandardPaths::Get().GetExecutablePath()));
+                               start ();
                        }
 
                        dcpomatic_sleep_seconds (1);
@@ -864,7 +861,7 @@ private:
 
                _film->write_metadata ();
 
-               if (send_to_other_tool (BATCH_JOB_PORT, bind (&start_batch_converter, _1), _film->directory()->string())) {
+               if (send_to_other_tool (BATCH_JOB_PORT, &start_batch_converter, _film->directory()->string())) {
                        error_dialog (this, _("Could not find batch converter."));
                }
        }
@@ -875,7 +872,7 @@ private:
                        return;
                }
 
-               if (send_to_other_tool (PLAYER_PLAY_PORT, bind (&start_player, _1), _film->dir(_film->dcp_name(false)).string())) {
+               if (send_to_other_tool (PLAYER_PLAY_PORT, &start_player, _film->dir(_film->dcp_name(false)).string())) {
                        error_dialog (this, _("Could not find player."));
                }
        }
@@ -905,9 +902,9 @@ private:
 
 
                dcp::LocalTime from (Config::instance()->signer_chain()->leaf().not_before());
-               from.add_minutes (1);
+               from.add_days (1);
                dcp::LocalTime to (Config::instance()->signer_chain()->leaf().not_after());
-               to.add_minutes (-1);
+               to.add_days (-1);
 
                optional<dcp::EncryptedKDM> kdm;
                try {
@@ -1314,7 +1311,15 @@ private:
                add_item (jobs_menu, _("Export...\tCtrl-E"), ID_jobs_export, NEEDS_FILM);
                jobs_menu->AppendSeparator ();
                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);
+
+#if defined(DCPOMATIC_OSX)
+               add_item (jobs_menu, _("S&how DCP in Finder"), ID_jobs_show_dcp, NEEDS_FILM | NOT_DURING_DCP_CREATION | NEEDS_CPL);
+#elif defined(DCPOMATIC_WINDOWS)
+               add_item (jobs_menu, _("S&how DCP in Explorer"), ID_jobs_show_dcp, NEEDS_FILM | NOT_DURING_DCP_CREATION | NEEDS_CPL);
+#else
+               add_item (jobs_menu, _("S&how DCP in Files"), ID_jobs_show_dcp, NEEDS_FILM | NOT_DURING_DCP_CREATION | NEEDS_CPL);
+#endif
+
                add_item (jobs_menu, _("Open DCP in &player"), ID_jobs_open_dcp_in_player, NEEDS_FILM | NOT_DURING_DCP_CREATION | NEEDS_CPL);
 
                wxMenu* view = new wxMenu;
@@ -1540,10 +1545,9 @@ private:
                        unsetenv ("UBUNTU_MENUPROXY");
 #endif
 
-#ifdef __WXOSX__
-                       ProcessSerialNumber serial;
-                       GetCurrentProcess (&serial);
-                       TransformProcessType (&serial, kProcessTransformToForegroundApplication);
+#ifdef DCPOMATIC_OSX
+                       dcpomatic_sleep_seconds (1);
+                       make_foreground_application ();
 #endif
 
                        dcpomatic_setup_path_encoding ();