diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2018-11-26 00:07:33 +0300 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2018-11-26 06:30:58 +0100 |
| commit | 761546e088dcccf3aa66c3f5c4853f110b93770f (patch) | |
| tree | 94dc648b78e78d685aab8b8b8e41b370a5b9db54 /svx | |
| parent | d3144a09dc5e2c21ffa34e3527bb6c94cd9926e8 (diff) | |
tdf#120703 PVS: V560 A part of conditional expression is always true/false
Change-Id: I1ac4c52ea51503373644bc58dcd4395c69f1a675
Reviewed-on: https://gerrit.libreoffice.org/64007
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'svx')
| -rw-r--r-- | svx/source/form/navigatortree.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/svx/source/form/navigatortree.cxx b/svx/source/form/navigatortree.cxx index c3f0aad652ce..f6aee2a8adc9 100644 --- a/svx/source/form/navigatortree.cxx +++ b/svx/source/form/navigatortree.cxx @@ -713,7 +713,7 @@ namespace svxform { // bHasHiddenControlsFormat means that only hidden controls are part of the data // hidden controls can be copied to a form only - if ( !_pTargetEntry || ( _pTargetEntry == m_pRootEntry ) || !IsFormEntry( _pTargetEntry ) ) + if ((_pTargetEntry == m_pRootEntry) || !IsFormEntry(_pTargetEntry)) return DND_ACTION_NONE; return bSelfSource ? ( DND_ACTION_COPYMOVE & _nAction ) : DND_ACTION_COPY; |