diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2019-02-08 17:41:36 +0300 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2019-02-11 07:08:50 +0100 |
| commit | 30c0134cd31c1409c374b9b0b38d2f2ac2f4bd03 (patch) | |
| tree | 5b5db3b64e6b9d601a508cdb427e2ca990fa72f2 /reportdesign | |
| parent | 04c1a98c10a24e65c4460d198c4009dc512c44ab (diff) | |
tdf#120703 PVS: V560 A part of conditional expression is always true/false
Change-Id: I4449fead67459bdcbdc0e9320129e7a5b36aecd9
Reviewed-on: https://gerrit.libreoffice.org/67545
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'reportdesign')
| -rw-r--r-- | reportdesign/source/ui/report/ViewsWindow.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/reportdesign/source/ui/report/ViewsWindow.cxx b/reportdesign/source/ui/report/ViewsWindow.cxx index 6e9c2e21a027..77fed84faa28 100644 --- a/reportdesign/source/ui/report/ViewsWindow.cxx +++ b/reportdesign/source/ui/report/ViewsWindow.cxx @@ -1581,10 +1581,10 @@ void OViewsWindow::handleKey(const vcl::KeyCode& _rCode) } } } - else + else // pHdl != nullptr { // move the handle - if ( pHdl && ( nX || nY ) ) + if (nX || nY) { const Point aStartPoint( pHdl->GetPos() ); const Point aEndPoint( pHdl->GetPos() + Point( nX, nY ) ); |