desktop/source/lib/init.cxx | 16 ----------------
1 file changed, 16 deletions(-)
New commits:
commit 81765aa741d14dd14eba598d402c138240b23efb
Author: Tor Lillqvist <t...@collabora.com>
AuthorDate: Fri Nov 2 23:39:44 2018 +0200
Commit: Tor Lillqvist <t...@collabora.com>
CommitDate: Sat Nov 3 15:31:46 2018 +0100
On iOS, lok_init_2() links directly to libreofficekit_hook_2()
Thus no need for any special __attribute__ ((visibility("default")))
dance.
Change-Id: I398d588a0165fd616ea0d259bbf2cab1e1a07ccb
Reviewed-on: https://gerrit.libreoffice.org/62796
Tested-by: Jenkins
Reviewed-by: Tor Lillqvist <t...@collabora.com>
diff --git a/desktop/source/lib/init.cxx b/desktop/source/lib/init.cxx
index fca76934088c..2496f9285c05 100644
--- a/desktop/source/lib/init.cxx
+++ b/desktop/source/lib/init.cxx
@@ -4251,19 +4251,7 @@ static int lo_initialize(LibreOfficeKit* pThis, const
char* pAppPath, const char
return bInitialized;
}
-// Undo our clever trick of having SAL_DLLPUBLIC_EXPORT actually not
-// meaning what is says in for the DISABLE_DYNLOADING case. See
-// <sal/types.h>. Normally, when building just one big dylib (Android)
-// or executable (iOS), most of our "public" symbols don't need to be
-// visible outside that resulting dylib/executable. But
-// libreofficekit_hook must be exported for dlsym() to find it,
-// though, at least on iOS.
-
-#if defined(__GNUC__) && defined(DISABLE_DYNLOADING)
-__attribute__ ((visibility("default")))
-#else
SAL_DLLPUBLIC_EXPORT
-#endif
LibreOfficeKit *libreofficekit_hook_2(const char* install_path, const char*
user_profile_url)
{
if (!gImpl)
@@ -4279,11 +4267,7 @@ LibreOfficeKit *libreofficekit_hook_2(const char*
install_path, const char* user
return static_cast<LibreOfficeKit*>(gImpl);
}
-#if defined(__GNUC__) && defined(DISABLE_DYNLOADING)
-__attribute__ ((visibility("default")))
-#else
SAL_DLLPUBLIC_EXPORT
-#endif
LibreOfficeKit *libreofficekit_hook(const char* install_path)
{
return libreofficekit_hook_2(install_path, nullptr);
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
