diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2018-11-13 00:40:43 +0300 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2018-11-18 12:32:18 +0100 |
| commit | a208ece66416048cdc99235a569bd8c928a07394 (patch) | |
| tree | e05fd7f828564bd370d21f897cfd2da7fa8f7488 /svtools | |
| parent | cf6b3bece7d46878302075329c72c7541c65bbd0 (diff) | |
tdf#120703 PVS: V560 A part of conditional expression is always true/false
Change-Id: I60bb778a88dd5619efd4858eb74d56d28616c777
Reviewed-on: https://gerrit.libreoffice.org/63520
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'svtools')
| -rw-r--r-- | svtools/source/control/ctrlbox.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/svtools/source/control/ctrlbox.cxx b/svtools/source/control/ctrlbox.cxx index 112ce7ea2da2..a5625a1a0001 100644 --- a/svtools/source/control/ctrlbox.cxx +++ b/svtools/source/control/ctrlbox.cxx @@ -155,14 +155,14 @@ long BorderWidthImpl::GuessWidth( long nLine1, long nLine2, long nGap ) double nWidth1 = lcl_getGuessedWidth( nLine1, m_nRate1, bLine1Change ); if ( bLine1Change ) aToCompare.push_back( nWidth1 ); - else if ( !bLine1Change && nWidth1 < 0 ) + else if (nWidth1 < 0) bInvalid = true; bool bLine2Change = bool( m_nFlags & BorderWidthImplFlags::CHANGE_LINE2 ); double nWidth2 = lcl_getGuessedWidth( nLine2, m_nRate2, bLine2Change ); if ( bLine2Change ) aToCompare.push_back( nWidth2 ); - else if ( !bLine2Change && nWidth2 < 0 ) + else if (nWidth2 < 0) bInvalid = true; bool bGapChange = bool( m_nFlags & BorderWidthImplFlags::CHANGE_DIST ); |