diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-09-08 09:26:12 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-01-11 20:42:21 +0100 |
| commit | 5467821e44bc31eeff617580e7c6d6649fdb9bed (patch) | |
| tree | 06d62622735d64f0942b38e3999de693d0ad704f /desktop | |
| parent | 154c72769d39258c5efbc35175824d08706a9314 (diff) | |
clang-tidy bugprone-unused-raii in RequestHandler
The message from clang-tidy is:
warning: object destroyed immediately after creation; did you mean
to name the object?
The guard in
RequestHandler::ExecuteCmdLineRequests
comes from
commit cf333a878ceed18d0343520a2c65be69fc433b1f
Date: Sun Jan 21 22:10:09 2018 +0300
tdf#38915: set cProcessed condition on any process outcome
and I'm sure it's intention was to set the flag on exit from the
function, not immediately.
Reviewed-on: https://gerrit.libreoffice.org/60183
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Tested-by: Jenkins
(cherry picked from commit 867ee21fe4c0c9216aea57850efec4de3c3fc554)
Change-Id: Ibf874a5774770df00b9db7f673554e7ffda55072
Reviewed-on: https://gerrit.libreoffice.org/66168
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'desktop')
| -rw-r--r-- | desktop/source/app/officeipcthread.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/desktop/source/app/officeipcthread.cxx b/desktop/source/app/officeipcthread.cxx index a0fabc9bdaf6..2e5596c259ff 100644 --- a/desktop/source/app/officeipcthread.cxx +++ b/desktop/source/app/officeipcthread.cxx @@ -1319,7 +1319,7 @@ bool RequestHandler::ExecuteCmdLineRequests( osl::ClearableMutexGuard aGuard( GetMutex() ); // ensure that Processed flag (if exists) is signaled in any outcome - ConditionSetGuard(aRequest.pcProcessed); + ConditionSetGuard aSetGuard(aRequest.pcProcessed); static std::vector<DispatchWatcher::DispatchRequest> aDispatchList; |