diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2020-02-18 17:19:38 +0100 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2020-02-18 20:02:07 +0100 |
| commit | 2ff77d088424bc3f5a8e38ef5c1aa3457d102eca (patch) | |
| tree | efdad0b7130d92a6525bb1c9d5816f4243460c57 /sfx2 | |
| parent | 5f4a65b7a3a29972c90a5ef4eb5fd7795b205cdf (diff) | |
SfxGlobalEvents_Impl::m_xEvents is never changed and never null
It is always set in the SfxGlobalEvents_Impl ctor with
m_xEvents = new GlobalEventConfig();
This code added with 3ddbfc311cd4471a6733b2d9dee338d3a80416c7 "INTEGRATION: CWS
fwkpostbeta03" appears to be nonsense boilerplate.
Change-Id: I6a6a75c619015815a8c6231862cd162de6361ecb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/88969
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'sfx2')
| -rw-r--r-- | sfx2/source/notify/globalevents.cxx | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/sfx2/source/notify/globalevents.cxx b/sfx2/source/notify/globalevents.cxx index 569ab3a98cb8..c1a1f674760c 100644 --- a/sfx2/source/notify/globalevents.cxx +++ b/sfx2/source/notify/globalevents.cxx @@ -355,15 +355,7 @@ void SfxGlobalEvents_Impl::implts_checkAndExecuteEventBindings(const document::D { try { - // SAFE -> - osl::ClearableMutexGuard aLock(m_aLock); - uno::Reference< container::XNameReplace > xEvents = m_xEvents; - aLock.clear(); - // <- SAFE - - uno::Any aAny; - if ( xEvents.is() && xEvents->hasByName( aEvent.EventName ) ) - aAny = xEvents->getByName(aEvent.EventName); + uno::Any aAny = m_xEvents->getByName(aEvent.EventName); SfxEvents_Impl::Execute(aAny, aEvent, nullptr); } catch ( uno::RuntimeException const & ) |