Fix some warnings

[lyx.git] / src / support / os_unix.cpp

diff --git a/src/support/os_unix.cpp b/src/support/os_unix.cpp
index b298a7da85efd444e5640baa6a7dff9e82637d49..3f16dd86563588e5e4ed6da5e6f952c9cf05f1db 100644 (file)
--- a/src/support/os_unix.cpp
+++ b/src/support/os_unix.cpp
@@ -23,10 +23,13 @@
 #include "support/lassert.h"
 
 #include <limits.h>
+#include <locale.h>
 #include <stdlib.h>
 
 #ifdef __APPLE__
 #include <CoreServices/CoreServices.h>
+#include <CoreFoundation/CoreFoundation.h>
+#include <CoreFoundation/CFArray.h>
 #endif
 
 using namespace std;
@@ -40,12 +43,12 @@ namespace {
 int argc_ = 0;
 char ** argv_ = 0;
 
-} // namespace anon
+} // namespace
 
-void init(int argc, char * argv[])
+void init(int argc, char ** argv[])
 {
        argc_ = argc;
-       argv_ = argv;
+       argv_ = *argv;
 
        // Set environment's default locale
        setlocale(LC_ALL, "");
@@ -190,23 +193,6 @@ string latex_path_list(string const & p)
 }
 
 
-bool is_valid_strftime(string const & p)
-{
-       string::size_type pos = p.find_first_of('%');
-       while (pos != string::npos) {
-               if (pos + 1 == string::npos)
-                       break;
-               if (!containsOnly(p.substr(pos + 1, 1),
-                       "aAbBcCdDeEFgGhHIjklmMnOpPrRsStTuUVwWxXyYzZ%+"))
-                       return false;
-               if (pos + 2 == string::npos)
-                     break;
-               pos = p.find_first_of('%', pos + 2);
-       }
-       return true;
-}
-
-
 char const * popen_read_mode()
 {
        return "r";
@@ -235,32 +221,46 @@ char path_separator(path_type)
 void windows_style_tex_paths(bool)
 {}
 
+
+#ifdef __APPLE__
+bool canAutoOpenFile(CFStringRef cfs_uti, LSRolesMask role)
+{
+       // Reference:
+       // https://developer.apple.com/reference/coreservices/1447734-lscopydefaultapplicationurlforco
+       CFURLRef outAppRef = LSCopyDefaultApplicationURLForContentType(cfs_uti, role, NULL);
+
+       if (outAppRef == NULL) return false;
+       CFRelease(outAppRef);
+       return true;
+}
+#endif
+
 bool canAutoOpenFile(string const & ext, auto_open_mode const mode)
 {
 #ifdef __APPLE__
-// Reference: http://developer.apple.com/documentation/Carbon/Reference/LaunchServicesReference/
+       // References:
+       // https://developer.apple.com/reference/coreservices/1447734-lscopydefaultapplicationurlforco
        CFStringRef cfs_ext = CFStringCreateWithBytes(kCFAllocatorDefault,
                                        (UInt8 *) ext.c_str(), ext.length(),
                                        kCFStringEncodingISOLatin1, false);
-       // this is what we would like to do but it seems that the
-       // viewer for PDF is often quicktime...
-       //LSRolesMask role = (mode == VIEW) ? kLSRolesViewer :  kLSRolesEditor;
-       (void)mode;
-       LSRolesMask role = kLSRolesAll;
-       FSRef outAppRef;
-       OSStatus status =
-               LSGetApplicationForInfo(kLSUnknownType, kLSUnknownCreator,
-                                       cfs_ext, role, &outAppRef, NULL);
+       CFStringRef cfs_uti = UTTypeCreatePreferredIdentifierForTag(kUTTagClassFilenameExtension, cfs_ext, NULL);
        CFRelease(cfs_ext);
+       if (cfs_uti == NULL) return false;
+
+       LSRolesMask role = (mode == VIEW) ? kLSRolesViewer :  kLSRolesEditor;
+       bool result = canAutoOpenFile(cfs_uti, role);
+       if (!result && mode == VIEW)
+               result = canAutoOpenFile(cfs_uti, kLSRolesEditor);
 
-       return status != kLSApplicationNotFoundErr;
+       CFRelease(cfs_uti);
+       return result;
 #else
        // silence compiler warnings
        (void)ext;
        (void)mode;
 
-       // currently, no default viewer is tried for non-windows system
-       // support for KDE/Gnome/Macintosh may be added later
+       // currently, no default viewer is tried for non-apple system
+       // support for KDE/Gnome may be added later
        return false;
 #endif
 }
@@ -271,51 +271,59 @@ bool autoOpenFile(string const & filename, auto_open_mode const mode,
 {
 #ifdef __APPLE__
 // Reference: http://developer.apple.com/documentation/Carbon/Reference/LaunchServicesReference/
-       FSRef fileref;
-       OSStatus status =
-               FSPathMakeRef((UInt8 *) filename.c_str(), &fileref, NULL);
-       if (status != 0)
-               return false;
-
        // this is what we would like to do but it seems that the
        // viewer for PDF is often quicktime...
        //LSRolesMask role = (mode == VIEW) ? kLSRolesViewer :  kLSRolesEditor;
        (void)mode;
-       LSRolesMask role = kLSRolesAll;
-       FSRef outAppRef;
+       LSRolesMask role = (mode == VIEW) ? kLSRolesAll :  kLSRolesEditor;
 
-       status = LSGetApplicationForItem(&fileref, role, &outAppRef, NULL);
-       if (status == kLSApplicationNotFoundErr)
-               return false;
+       CFURLRef docURL = CFURLCreateFromFileSystemRepresentation(
+               NULL, (UInt8 *) filename.c_str(), filename.size(), false);
+       CFURLRef appURL = LSCopyDefaultApplicationURLForURL(docURL, role, NULL);
+
+       CFURLRef docURLs[] = { docURL };
+       CFArrayRef launchItems = CFArrayCreate(
+               NULL,
+               (const void**)docURLs, sizeof(docURLs) / sizeof(CFURLRef),
+               NULL);
+       LSLaunchURLSpec launchUrlSpec = {
+               appURL, launchItems, NULL, kLSLaunchDefaults, NULL
+       };
 
        string const texinputs = os::latex_path_list(
-                       replaceCurdirPath(path, lyxrc.texinputs_prefix));
-       string const oldval = getEnv("TEXINPUTS");
-       string const newval = ".:" + texinputs + ":" + oldval;
-       if (!path.empty() && !lyxrc.texinputs_prefix.empty())
-               setEnv("TEXINPUTS", newval);
-
-       LSLaunchFSRefSpec inLaunchSpec;
-       inLaunchSpec.appRef = &outAppRef;
-       inLaunchSpec.numDocs = 1;
-       inLaunchSpec.itemRefs = &fileref;
-       inLaunchSpec.passThruParams = NULL;
-       inLaunchSpec.launchFlags = kLSLaunchDefaults;
-       inLaunchSpec.asyncRefCon = NULL;
-       status = LSOpenFromRefSpec(&inLaunchSpec, NULL);
-
-       if (!path.empty() && !lyxrc.texinputs_prefix.empty())
-               setEnv("TEXINPUTS", oldval);
-
-       return status != kLSApplicationNotFoundErr;
+               replaceCurdirPath(path, lyxrc.texinputs_prefix));
+       string const otherinputs = os::latex_path_list(path);
+       string const oldtexinputs = getEnv("TEXINPUTS");
+       string const newtexinputs = ".:" + texinputs + ":" + oldtexinputs;
+       string const oldbibinputs = getEnv("BIBINPUTS");
+       string const newbibinputs = ".:" + otherinputs + ":" + oldbibinputs;
+       string const oldbstinputs = getEnv("BSTINPUTS");
+       string const newbstinputs = ".:" + otherinputs + ":" + oldbstinputs;
+       string const oldtexfonts = getEnv("TEXFONTS");
+       string const newtexfonts = ".:" + otherinputs + ":" + oldtexfonts;
+       if (!path.empty() && !lyxrc.texinputs_prefix.empty()) {
+               setEnv("TEXINPUTS", newtexinputs);
+               setEnv("BIBINPUTS", newbibinputs);
+               setEnv("BSTINPUTS", newbstinputs);
+               setEnv("TEXFONTS", newtexfonts);
+       }
+       OSStatus const status = LSOpenFromURLSpec (&launchUrlSpec, NULL);
+       CFRelease(launchItems);
+       if (!path.empty() && !lyxrc.texinputs_prefix.empty()) {
+               setEnv("TEXINPUTS", oldtexinputs);
+               setEnv("BIBINPUTS", oldbibinputs);
+               setEnv("BSTINPUTS", oldbstinputs);
+               setEnv("TEXFONTS", oldtexfonts);
+       }
+       return status == 0;
 #else
        // silence compiler warnings
        (void)filename;
        (void)mode;
        (void)path;
 
-       // currently, no default viewer is tried for non-windows system
-       // support for KDE/Gnome/Macintosh may be added later
+       // currently, no default viewer is tried for non-apple system
+       // support for KDE/Gnome may be added later
        return false;
 #endif
 }