diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2016-04-04 17:50:31 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2016-04-04 17:50:31 +0200 |
commit | e962fa79bbbfe3b596b06838596c288d99ed3d4f (patch) | |
tree | 35e52f614491627222ef4f6983fe39439e4bd15f /desktop | |
parent | 5da71ff3b088f224cd3a731ce51c620e0cd3e5d3 (diff) |
-Werror=shadow
Change-Id: I231396660573e395ef7f949687c6e536a8a60b20
Diffstat (limited to 'desktop')
-rw-r--r-- | desktop/source/app/officeipcthread.cxx | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/desktop/source/app/officeipcthread.cxx b/desktop/source/app/officeipcthread.cxx index a5d9ac7b6f5f..ba85d6430f62 100644 --- a/desktop/source/app/officeipcthread.cxx +++ b/desktop/source/app/officeipcthread.cxx @@ -373,7 +373,7 @@ protected: virtual ~IpcThread() {} - bool process(OString const & arguments, bool * wait); + bool process(OString const & arguments, bool * waitProcessed); RequestHandler * handler_; }; @@ -595,14 +595,14 @@ void DbusIpcThread::execute() } char const * argstr; dbus_message_iter_get_basic(&it, &argstr); - bool wait = false; + bool waitProcessed = false; { osl::MutexGuard g(RequestHandler::GetMutex()); - if (!process(argstr, &wait)) { + if (!process(argstr, &waitProcessed)) { continue; } } - if (wait) { + if (waitProcessed) { handler_->cProcessed.wait(); } DbusMessageHolder repl(dbus_message_new_method_return(msg.message)); @@ -941,8 +941,8 @@ void RequestHandler::WaitForReady() } } -bool IpcThread::process(OString const & arguments, bool * wait) { - assert(wait != nullptr); +bool IpcThread::process(OString const & arguments, bool * waitProcessed) { + assert(waitProcessed != nullptr); std::unique_ptr< CommandLineArgs > aCmdLineArgs; try @@ -1140,7 +1140,7 @@ bool IpcThread::process(OString const & arguments, bool * wait) { ImplPostForeignAppEvent( pAppEvent ); } } - *wait = bDocRequestSent; + *waitProcessed = bDocRequestSent; return true; } @@ -1188,15 +1188,15 @@ void PipeIpcThread::execute() if (aArguments.isEmpty()) continue; - bool wait = false; - if (!process(aArguments, &wait)) { + bool waitProcessed = false; + if (!process(aArguments, &waitProcessed)) { continue; } // we don't need the mutex any longer... aGuard.clear(); // wait for processing to finish - if (wait) + if (waitProcessed) handler_->cProcessed.wait(); // processing finished, inform the requesting end: n = aStreamPipe.write( |