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

Stephan Bergmann sbergman at redhat.com
Fri Jan 19 07:20:25 UTC 2018


 basctl/source/basicide/basobj3.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit f867ead158730727e5eff0917a037ca0051d9df9
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Thu Jan 18 17:33:00 2018 +0100

    Fix callback function type
    
    ...that got broken with c394363c84064c041391627602665ea1fa74db60
    "loplugin:constparams in basctl", causing -fsanitize=function to complain:
    
    > sfx2/source/appl/app.cxx:452:28: runtime error: call to function basicide_handle_basic_error through pointer to incorrect function type 'long (*)(void *)'
    > basctl/source/basicide/basobj3.cxx:48: note: basicide_handle_basic_error defined here
    >     #0 0x7ff6340234aa in SfxApplication::GlobalBasicErrorHdl_Impl(SfxApplication*, StarBASIC*) sfx2/source/appl/app.cxx:452:28
    >     #1 0x7ff63401cbef in SfxApplication::LinkStubGlobalBasicErrorHdl_Impl(void*, StarBASIC*) sfx2/source/appl/app.cxx:434:1
    >     #2 0x7ff639142148 in Link<StarBASIC*, bool>::Call(StarBASIC*) const include/tools/link.hxx:84:45
    >     #3 0x7ff63911f350 in StarBASIC::RTError(ErrCode, rtl::OUString const&, int, int, int) basic/source/classes/sb.cxx:1681:37
    >     #4 0x7ff63965aafe in SbiInstance::Abort() basic/source/runtime/runtime.cxx:518:16
    >     #5 0x7ff639664363 in SbiRuntime::Step() basic/source/runtime/runtime.cxx:865:28
    >     #6 0x7ff6392ec321 in SbModule::Run(SbMethod*) basic/source/classes/sbxmod.cxx:1141:25
    [...]
    
    Change-Id: I66bd1ca720cebca4fb5fd52d86920de3fc7a604e
    Reviewed-on: https://gerrit.libreoffice.org/48142
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Stephan Bergmann <sbergman at redhat.com>

diff --git a/basctl/source/basicide/basobj3.cxx b/basctl/source/basicide/basobj3.cxx
index 7043bba6110a..246486331538 100644
--- a/basctl/source/basicide/basobj3.cxx
+++ b/basctl/source/basicide/basobj3.cxx
@@ -44,7 +44,7 @@ using namespace ::com::sun::star::uno;
 using namespace ::com::sun::star::container;
 
 extern "C" {
-    SAL_DLLPUBLIC_EXPORT long basicide_handle_basic_error( void const * pPtr )
+    SAL_DLLPUBLIC_EXPORT long basicide_handle_basic_error( void * pPtr )
     {
         return HandleBasicError( static_cast<StarBASIC const *>(pPtr) );
     }


More information about the Libreoffice-commits mailing list