X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2Fsupport%2Fos_unix.cpp;h=34cfd478a61650ce9bcb0beec5a204886bd34dee;hb=e8c1cb5074e36a344553d57537f8177345a86686;hp=135d3149916eb597ac18fd6b5a06e0cfefd1efe2;hpb=031748d9c88c60d6a7c79b00599955c49a3c56aa;p=lyx.git

diff --git a/src/support/os_unix.cpp b/src/support/os_unix.cpp
index 135d314991..34cfd478a6 100644
--- a/src/support/os_unix.cpp
+++ b/src/support/os_unix.cpp
@@ -43,7 +43,7 @@ namespace {
 int argc_ = 0;
 char ** argv_ = 0;
 
-} // namespace anon
+} // namespace
 
 void init(int argc, char ** argv[])
 {
@@ -193,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";
@@ -238,46 +221,32 @@ 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__
-	// References:
-	// https://developer.apple.com/reference/coreservices/1447734-lscopydefaultapplicationurlforco
+// Reference: http://developer.apple.com/documentation/Carbon/Reference/LaunchServicesReference/
 	CFStringRef cfs_ext = CFStringCreateWithBytes(kCFAllocatorDefault,
 					(UInt8 *) ext.c_str(), ext.length(),
 					kCFStringEncodingISOLatin1, false);
-	CFStringRef cfs_uti = UTTypeCreatePreferredIdentifierForTag(kUTTagClassFilenameExtension, cfs_ext, NULL);
+	// 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);
 	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);
-
-	CFRelease(cfs_uti);
-	return result;
+	return status != kLSApplicationNotFoundErr;
 #else
 	// silence compiler warnings
 	(void)ext;
 	(void)mode;
 
-	// currently, no default viewer is tried for non-apple system
-	// support for KDE/Gnome may be added later
+	// currently, no default viewer is tried for non-windows system
+	// support for KDE/Gnome/Macintosh may be added later
 	return false;
 #endif
 }
@@ -292,11 +261,14 @@ bool autoOpenFile(string const & filename, auto_open_mode const mode,
 	// viewer for PDF is often quicktime...
 	//LSRolesMask role = (mode == VIEW) ? kLSRolesViewer :  kLSRolesEditor;
 	(void)mode;
-	LSRolesMask role = (mode == VIEW) ? kLSRolesAll :  kLSRolesEditor;
+	LSRolesMask role = kLSRolesAll;
 
 	CFURLRef docURL = CFURLCreateFromFileSystemRepresentation(
 		NULL, (UInt8 *) filename.c_str(), filename.size(), false);
-	CFURLRef appURL = LSCopyDefaultApplicationURLForURL(docURL, role, NULL);
+	CFURLRef appURL;
+	OSStatus status = LSGetApplicationForURL(docURL, role, NULL, &appURL);
+	if (status == kLSApplicationNotFoundErr)
+		return false;
 
 	CFURLRef docURLs[] = { docURL };
 	CFArrayRef launchItems = CFArrayCreate(
@@ -324,7 +296,7 @@ bool autoOpenFile(string const & filename, auto_open_mode const mode,
 		setEnv("BSTINPUTS", newbstinputs);
 		setEnv("TEXFONTS", newtexfonts);
 	}
-	OSStatus const status = LSOpenFromURLSpec (&launchUrlSpec, NULL);
+	status = LSOpenFromURLSpec (&launchUrlSpec, NULL);
 	CFRelease(launchItems);
 	if (!path.empty() && !lyxrc.texinputs_prefix.empty()) {
 		setEnv("TEXINPUTS", oldtexinputs);
@@ -339,8 +311,8 @@ bool autoOpenFile(string const & filename, auto_open_mode const mode,
 	(void)mode;
 	(void)path;
 
-	// currently, no default viewer is tried for non-apple system
-	// support for KDE/Gnome may be added later
+	// currently, no default viewer is tried for non-windows system
+	// support for KDE/Gnome/Macintosh may be added later
 	return false;
 #endif
 }