strepsirrhini/office-gobmx
9
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

PVS: V560 A part of conditional expression is always true: !bSingle

Browse source 
Since
commit 5a725d1fde
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
This commit is contained in:
Xisco Fauli 2024-11-27 11:56:46 +01:00
parent 69a020c19a
commit e196718795
1 changed files with 3 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
sc/source/ui/view/output.cxx
View file

@ -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)