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 /vcl/source/window/menu.cxx | |
| 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 'vcl/source/window/menu.cxx')
| -rw-r--r-- | vcl/source/window/menu.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/vcl/source/window/menu.cxx b/vcl/source/window/menu.cxx index b9833ae4a503..45532cd448a8 100644 --- a/vcl/source/window/menu.cxx +++ b/vcl/source/window/menu.cxx @@ -629,7 +629,7 @@ sal_uInt16 Menu::ImplGetPrevVisible( sal_uInt16 nPos ) const { for ( size_t n = nPos; n; ) { - if ( n && ImplIsVisible( --n ) ) + if (ImplIsVisible(--n)) return n; } return ITEMPOS_INVALID; |