summaryrefslogtreecommitdiff
path: root/shell
diff options
context:
space:
mode:
authorStephan Bergmann <sbergman@redhat.com>2017-12-14 08:45:02 +0100
committerStephan Bergmann <sbergman@redhat.com>2017-12-14 13:03:06 +0100
commit6f4f5677acff2711b4cfdae6ba1a0892e62b179a (patch)
treeba97fc99b67b9b139bdf6807ceedcb168fa68fd2 /shell
parent581640ac53f8568157ffc3b5c82b2ae7da576e3b (diff)
No need to keep these whitelisted functions decorated with SAL_CALL
The only effect SAL_CALL effectively has on LO-internal code is to change non- static member functions from __thiscall to __cdecl in MSVC (where all other functions are __cdecl by default, anyway). (For 3rd-party code, it could be argued that SAL_CALL is useful on function declarations in the URE stable interface other than non-static member functions, too, in case 3rd-party code uses a compiler switch to change the default calling convention to something other than __cdecl. But loplugin:salcall exempts the URE stable interface, anyway.) One could argue that SAL_CALL, even if today it effectively only affects non- static member functions in MSVC, could be extended in the future to affect more functions on more platforms. However, the current code would already not support that. For example, 3af500580b1c82eabd60335c9ebc458a3f68850c "loplugin:salcall fix functions" changed FrameControl_createInstance in UnoControls/source/base/registercontrols.cxx to no longer be SAL_CALL, even though its address (in ctl_component_getFacrory, in the same file) is passed to cppuhelper::createSingleFactory as an argument of type cppu::ComponentInstantiation, which is a pointer to SAL_CALL function. Change-Id: I3acbf7314a3d7868ed70e35bb5c47bc11a0b7ff6 Reviewed-on: https://gerrit.libreoffice.org/46436 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'shell')
-rw-r--r--shell/source/backends/localebe/localebackend.cxx4
-rw-r--r--shell/source/backends/localebe/localebackend.hxx4
2 files changed, 4 insertions, 4 deletions
diff --git a/shell/source/backends/localebe/localebackend.cxx b/shell/source/backends/localebe/localebackend.cxx
index 3392bb73cfc0..056d8a48bb32 100644
--- a/shell/source/backends/localebe/localebackend.cxx
+++ b/shell/source/backends/localebe/localebackend.cxx
@@ -320,7 +320,7 @@ css::uno::Any LocaleBackend::getPropertyValue(
}
-OUString SAL_CALL LocaleBackend::getBackendName() {
+OUString LocaleBackend::getBackendName() {
return OUString("com.sun.star.comp.configuration.backend.LocaleBackend") ;
}
@@ -329,7 +329,7 @@ OUString SAL_CALL LocaleBackend::getImplementationName()
return getBackendName() ;
}
-uno::Sequence<OUString> SAL_CALL LocaleBackend::getBackendServiceNames()
+uno::Sequence<OUString> LocaleBackend::getBackendServiceNames()
{
uno::Sequence<OUString> aServiceNameList { "com.sun.star.configuration.backend.LocaleBackend" };
return aServiceNameList ;
diff --git a/shell/source/backends/localebe/localebackend.hxx b/shell/source/backends/localebe/localebackend.hxx
index 4fd0e75b593b..006d669db0ee 100644
--- a/shell/source/backends/localebe/localebackend.hxx
+++ b/shell/source/backends/localebe/localebackend.hxx
@@ -56,13 +56,13 @@ class LocaleBackend final : public ::cppu::WeakImplHelper <
@return implementation name
*/
- static OUString SAL_CALL getBackendName() ;
+ static OUString getBackendName() ;
/**
Provides the supported services names
@return service names
*/
- static uno::Sequence<OUString> SAL_CALL getBackendServiceNames() ;
+ static uno::Sequence<OUString> getBackendServiceNames() ;
// XPropertySet
virtual css::uno::Reference< css::beans::XPropertySetInfo > SAL_CALL