From 17ddc78c2d0ff1b2af82cb8e8ed9c5db3f959d19 Mon Sep 17 00:00:00 2001
From: Tor Lillqvist <tml@collabora.com>
Date: Tue, 19 Jun 2018 10:02:42 +0300
Subject: Don't waste time in macxp_resolveAlias() on files inside the app
 bundle

LibreOffice uses its low-level API to look up hundreds of files inside
the app bundle all the time, and especially when starting as a whole,
or when starting specific aspects of the application (like after
typing a first character into a Writer document in a session). Having
all those go through this alias or bookmark dance is just insane.
There won't be any there.

This shaves almost a second from the delay after typing the first
character into a Writer document in a session. There is still a
noticeable delay left, though, likely mostly caused by Python
(Lightproof) initialisation slowness. (It's cross-platform.)

I would say that it is a bit questionable whether the
macxp_resolveAlias() functionality is worth it at all, even.

Change-Id: I2461141c6b58738befd0db4902eb25e63b788b79
Reviewed-on: https://gerrit.libreoffice.org/56081
Reviewed-by: Andras Timar <andras.timar@collabora.com>
Tested-by: Andras Timar <andras.timar@collabora.com>
---
 sal/Library_sal.mk     |  5 +++--
 sal/osl/unx/system.cxx | 13 +++++++++++++
 sal/osl/unx/system.mm  |  1 +
 3 files changed, 17 insertions(+), 2 deletions(-)
 create mode 100644 sal/osl/unx/system.mm

(limited to 'sal')

diff --git a/sal/Library_sal.mk b/sal/Library_sal.mk
index 17d89ae79e4a..b0921dd8ff24 100644
--- a/sal/Library_sal.mk
+++ b/sal/Library_sal.mk
@@ -178,21 +178,22 @@ $(eval $(call gb_Library_add_exception_objects,sal,\
 	sal/osl/unx/security \
 	sal/osl/unx/signal \
 	sal/osl/unx/socket \
-	sal/osl/unx/system \
 	sal/osl/unx/tempfile \
 	sal/osl/unx/thread \
 	sal/osl/unx/time \
 	$(if $(filter-out ANDROID IOS,$(OS)), sal/osl/unx/salinit) \
 ))
 
-# Note that the uunxapi.mm file just includes the uunxapi.cxx one
+# Note that the uunxapi.mm file just includes the uunxapi.cxx one. Ditto for system.mm
 ifeq ($(OS),MACOSX)
 $(eval $(call gb_Library_add_objcxxobjects,sal,\
 	sal/osl/unx/uunxapi \
+	sal/osl/unx/system \
 ))
 else
 $(eval $(call gb_Library_add_exception_objects,sal,\
 	sal/osl/unx/uunxapi \
+	sal/osl/unx/system \
 ))
 endif
 
diff --git a/sal/osl/unx/system.cxx b/sal/osl/unx/system.cxx
index 7003b4ff5a41..9b7f05d93b28 100644
--- a/sal/osl/unx/system.cxx
+++ b/sal/osl/unx/system.cxx
@@ -37,6 +37,10 @@
 #define RTL_MUTEX_LOCK
 #define RTL_MUTEX_UNLOCK
 
+#include <premac.h>
+#include <Foundation/Foundation.h>
+#include <postmac.h>
+
 #else //defined(MACOSX)
 
 static pthread_mutex_t getrtl_mutex = PTHREAD_MUTEX_INITIALIZER;
@@ -162,6 +166,15 @@ int macxp_resolveAlias(char *path, int buflen)
   CFErrorRef cferror;
   CFDataRef cfbookmark;
 
+  // Don't even try anything for files inside the app bundle. Just a
+  // waste of time.
+
+  static const char * const appBundle = [[[NSBundle mainBundle] bundlePath] UTF8String];
+
+  const size_t appBundleLen = strlen(appBundle);
+  if (strncmp(path, appBundle, appBundleLen) == 0 && path[appBundleLen] == '/')
+      return 0;
+
   char *unprocessedPath = path;
 
   if ( *unprocessedPath == '/' )
diff --git a/sal/osl/unx/system.mm b/sal/osl/unx/system.mm
new file mode 100644
index 000000000000..7495a3e73058
--- /dev/null
+++ b/sal/osl/unx/system.mm
@@ -0,0 +1 @@
+#include "system.cxx"
-- 
cgit