[Libreoffice-commits] core.git: sfx2/source

Tomoyuki Kubota (via logerrit) logerrit at kemper.freedesktop.org
Sun Apr 12 10:33:22 UTC 2020


 sfx2/source/appl/app.cxx |   31 +++++++++++++++++++++++++++----
 1 file changed, 27 insertions(+), 4 deletions(-)

New commits:
commit 7768889c351e079f4fc59647ccafc37013217040
Author:     Tomoyuki Kubota <himajin100000 at gmail.com>
AuthorDate: Sun Apr 12 17:27:24 2020 +0900
Commit:     Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Sun Apr 12 12:32:49 2020 +0200

    More fixes to loading of basctl with --enable-mergelibs
    
    Change-Id: I3458110f0cd7a5ff9e81d59d864f831038335ad2
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92077
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>

diff --git a/sfx2/source/appl/app.cxx b/sfx2/source/appl/app.cxx
index c09b415825f6..5bb09cce532b 100644
--- a/sfx2/source/appl/app.cxx
+++ b/sfx2/source/appl/app.cxx
@@ -18,6 +18,7 @@
  */
 
 #include <config_feature_desktop.h>
+#include <config_options.h>
 #include <sal/log.hxx>
 #include <osl/module.hxx>
 #include <tools/debug.hxx>
@@ -390,9 +391,20 @@ IMPL_STATIC_LINK( SfxApplication, GlobalBasicErrorHdl_Impl, StarBASIC*, pStarBas
 #else
 
 #ifndef DISABLE_DYNLOADING
-    // load basctl module
     osl::Module aMod;
-    aMod.loadRelative(&thisModule, SVLIBRARY("basctl"));
+    // load basctl module
+    if (!aMod.loadRelative(
+            &thisModule,
+#if ENABLE_MERGELIBS
+            SVLIBRARY("merged")
+#else
+            SVLIBRARY("basctl")
+#endif
+        ))
+    {
+        SAL_WARN("sfx.appl", "cannot load basctl");
+        return false;
+    }
 
     // get symbol
     basicide_handle_basic_error pSymbol = reinterpret_cast<basicide_handle_basic_error>(aMod.getFunctionSymbol("basicide_handle_basic_error"));
@@ -483,9 +495,20 @@ void SfxApplication::MacroOrganizer(weld::Window* pParent, sal_Int16 nTabId)
 #else
 
 #ifndef DISABLE_DYNLOADING
-    // load basctl module
     osl::Module aMod;
-    aMod.loadRelative(&thisModule, SVLIBRARY("basctl"));
+    // load basctl module
+    if (!aMod.loadRelative(
+            &thisModule,
+#if ENABLE_MERGELIBS
+            SVLIBRARY("merged")
+#else
+            SVLIBRARY("basctl")
+#endif
+        ))
+    {
+        SAL_WARN("sfx.appl", "cannot load basctl");
+        return;
+    }
 
     // get symbol
     basicide_macro_organizer pSymbol = reinterpret_cast<basicide_macro_organizer>(aMod.getFunctionSymbol("basicide_macro_organizer"));


More information about the Libreoffice-commits mailing list