[Libreoffice-commits] core.git: desktop/source
Stephan Bergmann
sbergman at redhat.com
Tue Mar 22 16:32:25 UTC 2016
desktop/source/app/officeipcthread.cxx | 4 ----
1 file changed, 4 deletions(-)
New commits:
commit 54dae86076d01306ecfe74fa21d577d82332ab3a
Author: Stephan Bergmann <sbergman at redhat.com>
Date: Tue Mar 22 17:25:26 2016 +0100
~OfficeIPCThread can only be reached with pGlobalOfficeIPCThread cleared
...as only one instance of OfficeIPCThread is ever created (in
OfficeIPCThread::EnableOfficeIPCThread), which is then either assigned to
pGlobalOfficeIPCThread (if this is the first soffice instance), or
pGlobalOfficeIPCThread is left unassigned and the OfficeIPCThread instance goes
out of scope at the end of EnableOfficeIPCThread (if another soffice instance is
already running).
So mpDispatchWatcher also does not need to be cleared here, as it is only ever
accessed through pGlobalOfficeIPCThread. And thus, there is no longer a need to
lock GetMutex() here, either.
Change-Id: I56cffc43361d01d747084153f318b8488519cf9b
diff --git a/desktop/source/app/officeipcthread.cxx b/desktop/source/app/officeipcthread.cxx
index 9c6fab9..42d89cc 100644
--- a/desktop/source/app/officeipcthread.cxx
+++ b/desktop/source/app/officeipcthread.cxx
@@ -653,11 +653,7 @@ OfficeIPCThread::OfficeIPCThread() :
OfficeIPCThread::~OfficeIPCThread()
{
- ::osl::ClearableMutexGuard aGuard( GetMutex() );
-
- mpDispatchWatcher.clear();
maPipe.close();
- pGlobalOfficeIPCThread.clear();
}
void OfficeIPCThread::SetReady()
More information about the Libreoffice-commits
mailing list