diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2019-02-07 14:40:49 +0300 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2019-02-11 07:08:08 +0100 |
| commit | 3165146f6deef86eb408837275ec3f7edda0183b (patch) | |
| tree | ca899778648ab35eccfd6b520629b293db83ceeb /toolkit | |
| parent | 774cc7e0930f223fded1a774e0b71e6b68d48f5f (diff) | |
tdf#120703 PVS: V560 A part of conditional expression is always true/false
Change-Id: I62cb3b8927d664b3d5359ee6ac7db30d354f4821
Reviewed-on: https://gerrit.libreoffice.org/67496
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'toolkit')
| -rw-r--r-- | toolkit/source/awt/vclxmenu.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/toolkit/source/awt/vclxmenu.cxx b/toolkit/source/awt/vclxmenu.cxx index d9ec3a6487f7..716eec4a0105 100644 --- a/toolkit/source/awt/vclxmenu.cxx +++ b/toolkit/source/awt/vclxmenu.cxx @@ -303,7 +303,7 @@ void VCLXMenu::removeItem( return; sal_Int32 nItemCount = static_cast<sal_Int32>(mpMenu->GetItemCount()); - if ( ( nCount > 0 ) && ( nPos >= 0 ) && ( nPos < nItemCount ) && ( nItemCount > 0 )) + if ((nCount > 0) && (nPos >= 0) && (nPos < nItemCount)) { sal_Int16 nP = sal::static_int_cast< sal_Int16 >( std::min( static_cast<int>(nPos+nCount), static_cast<int>(nItemCount) )); |