diff options
| author | Xisco Fauli <xiscofauli@libreoffice.org> | 2024-11-27 11:56:46 +0100 |
|---|---|---|
| committer | Xisco Fauli <xiscofauli@libreoffice.org> | 2024-11-27 13:22:00 +0100 |
| commit | e1967187957e1e4b56962564a81917d0ba92cac0 (patch) | |
| tree | afecec8f453bd036c4ffdb8d01e662dd4e6d52e4 /sc/source | |
| parent | 69a020c19a5f1a8abe8a024abb73c862be2a762d (diff) | |
PVS: V560 A part of conditional expression is always true: !bSingle
Since
commit 5a725d1fde7381ef9c02afcb639235a11e52182c
Author: Noel Grandin <noel.grandin@collabora.co.uk>
Date: Tue Mar 28 15:27:36 2017 +0200
loplugin:singlevalfields
Change-Id: I5fabc0537f385531fedb34a0becf69e480d39394
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/177404
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
Tested-by: Jenkins
Diffstat (limited to 'sc/source')
| -rw-r--r-- | sc/source/ui/view/output.cxx | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/sc/source/ui/view/output.cxx b/sc/source/ui/view/output.cxx index 2fa575982783..ddf3dd5e0b78 100644 --- a/sc/source/ui/view/output.cxx +++ b/sc/source/ui/view/output.cxx @@ -361,7 +361,6 @@ void ScOutputData::DrawGrid(vcl::RenderContext& rRenderContext, bool bGrid, bool ScBreakType nBreak = ScBreakType::NONE; ScBreakType nBreakOld = ScBreakType::NONE; - bool bSingle; bool bDashed = false; Color aPageColor; Color aManualColor; @@ -469,8 +468,8 @@ void ScOutputData::DrawGrid(vcl::RenderContext& rRenderContext, bool bGrid, bool bool bDraw = bGrid || nBreakOld != ScBreakType::NONE || bMergeCover; // simple grid only if set that way sal_uInt16 nWidthXplus1 = pRowInfo[0].basicCellInfo(nX+1).nWidth; - bSingle = false; //! get into Fillinfo !!!!! - if ( nX<mpDoc->MaxCol() && !bSingle ) + bool bSingle = false; //! get into Fillinfo !!!!! + if ( nX<mpDoc->MaxCol() ) { bSingle = ( nWidthXplus1 == 0 ); for (nArrY=1; nArrY+1<nArrCount && !bSingle; nArrY++) @@ -586,7 +585,7 @@ void ScOutputData::DrawGrid(vcl::RenderContext& rRenderContext, bool bGrid, bool bool bDraw = bGrid || nBreakOld != ScBreakType::NONE || bMergeCover; // simple grid only if set so bool bNextYisNextRow = (pRowInfo[nArrYplus1].nRowNo == nYplus1); - bSingle = !bNextYisNextRow; // Hidden + bool bSingle = !bNextYisNextRow; // Hidden for (SCCOL i=nX1; i<=nX2 && !bSingle; i++) { if (pRowInfo[nArrYplus1].cellInfo(i).bVOverlapped) |