diff options
author | Noel Grandin <noel@peralex.com> | 2016-03-01 15:39:39 +0200 |
---|---|---|
committer | Noel Grandin <noel@peralex.com> | 2016-03-02 07:57:38 +0200 |
commit | 09b5fe4c898cacd69042d1c585e27daf5d86cae7 (patch) | |
tree | ec6a77f4d3d2c595da0ef1f61721b23006246109 /desktop/source/app/officeipcthread.cxx | |
parent | b0e18a68c86733b6d88a0c990d4fa4d938680727 (diff) |
loplugin:unuseddefaultparam in desktop
Change-Id: I9a7ac03dcbc3849eced0f8431e186b59b31b2418
Diffstat (limited to 'desktop/source/app/officeipcthread.cxx')
-rw-r--r-- | desktop/source/app/officeipcthread.cxx | 26 |
1 files changed, 10 insertions, 16 deletions
diff --git a/desktop/source/app/officeipcthread.cxx b/desktop/source/app/officeipcthread.cxx index a3505795d3fa..2a726277c4b6 100644 --- a/desktop/source/app/officeipcthread.cxx +++ b/desktop/source/app/officeipcthread.cxx @@ -400,7 +400,7 @@ void OfficeIPCThread::SetDowning() static bool s_bInEnableRequests = false; -void OfficeIPCThread::EnableRequests( bool i_bEnable ) +void OfficeIPCThread::EnableRequests() { // switch between just queueing the requests and executing them ::osl::MutexGuard aGuard( GetMutex() ); @@ -408,14 +408,11 @@ void OfficeIPCThread::EnableRequests( bool i_bEnable ) if ( pGlobalOfficeIPCThread.is() ) { s_bInEnableRequests = true; - pGlobalOfficeIPCThread->mbRequestsEnabled = i_bEnable; - if( i_bEnable ) - { - // hit the compiler over the head - ProcessDocumentsRequest aEmptyReq = ProcessDocumentsRequest( boost::optional< OUString >() ); - // trigger already queued requests - OfficeIPCThread::ExecuteCmdLineRequests( aEmptyReq ); - } + pGlobalOfficeIPCThread->mbRequestsEnabled = true; + // hit the compiler over the head + ProcessDocumentsRequest aEmptyReq = ProcessDocumentsRequest( boost::optional< OUString >() ); + // trigger already queued requests + OfficeIPCThread::ExecuteCmdLineRequests( aEmptyReq ); s_bInEnableRequests = false; } } @@ -430,14 +427,14 @@ bool OfficeIPCThread::AreRequestsPending() return false; } -void OfficeIPCThread::RequestsCompleted( int nCount ) +void OfficeIPCThread::RequestsCompleted() { // Remove nCount pending requests from our internal counter ::osl::MutexGuard aGuard( GetMutex() ); if ( pGlobalOfficeIPCThread.is() ) { if ( pGlobalOfficeIPCThread->mnPendingRequests > 0 ) - pGlobalOfficeIPCThread->mnPendingRequests -= nCount; + pGlobalOfficeIPCThread->mnPendingRequests --; } } @@ -682,12 +679,9 @@ void OfficeIPCThread::SetReady( } } -void OfficeIPCThread::WaitForReady( - rtl::Reference< OfficeIPCThread > const & pThread) - +void OfficeIPCThread::WaitForReady() { - rtl::Reference< OfficeIPCThread > const & t( - pThread.is() ? pThread : pGlobalOfficeIPCThread); + rtl::Reference< OfficeIPCThread > const & t(pGlobalOfficeIPCThread); if (t.is()) { t->cReady.wait(); |