diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2018-11-26 10:36:52 +0300 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2018-12-01 18:20:24 +0100 |
| commit | fff501a3393b459c512ec155e2d2cd935e7885a2 (patch) | |
| tree | 625ea119195475e2680ba7f1be1c73b2a31a0684 /sw | |
| parent | af843af4c816ef25246c815e187729ec25b2f000 (diff) | |
tdf#120703 PVS: V560 A part of conditional expression is always true/false
Change-Id: I38f3e3243f8cb891c9a0fe2f4c68bc76acd1ef44
Reviewed-on: https://gerrit.libreoffice.org/64020
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'sw')
| -rw-r--r-- | sw/source/core/text/porlay.cxx | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sw/source/core/text/porlay.cxx b/sw/source/core/text/porlay.cxx index 02900e9e5d79..781c8aef5302 100644 --- a/sw/source/core/text/porlay.cxx +++ b/sw/source/core/text/porlay.cxx @@ -1510,7 +1510,7 @@ sal_Int32 SwScriptInfo::MaskHiddenRanges( const SwTextNode& rNode, OUStringBuffe while ( nHiddenStart < nHiddenEnd && nHiddenStart < nEnd ) { - if ( nHiddenStart >= nStt && nHiddenStart < nEnd ) + if (nHiddenStart >= nStt) { rText[nHiddenStart] = cChar; ++nNumOfHiddenChars; @@ -1605,7 +1605,7 @@ bool SwScriptInfo::GetBoundsOfHiddenRange( const SwTextNode& rNode, sal_Int32 nP if ( nHiddenStart > nPos ) break; - if ( nHiddenStart <= nPos && nPos < nHiddenEnd ) + if (nPos < nHiddenEnd) { rnStartPos = nHiddenStart; rnEndPos = std::min<sal_Int32>(nHiddenEnd, @@ -1644,7 +1644,7 @@ bool SwScriptInfo::GetBoundsOfHiddenRange(TextFrameIndex nPos, if ( nHiddenStart > nPos ) break; - if ( nHiddenStart <= nPos && nPos < nHiddenEnd ) + if (nPos < nHiddenEnd) { rnStartPos = nHiddenStart; rnEndPos = nHiddenEnd; |