summaryrefslogtreecommitdiff
path: root/sw/source
diff options
context:
space:
mode:
authorStephan Bergmann <sbergman@redhat.com>2021-06-08 17:23:30 +0200
committerNoel Grandin <noel.grandin@collabora.co.uk>2021-06-09 20:44:38 +0200
commita74dab375e894c3b87f5279cd7c43c26e4c79bb4 (patch)
tree5cb14b29baf01e7dc6297e271311af2c845b4da3 /sw/source
parentaa39400c73fac4b0430ee47a0d18bf4978f7f2d0 (diff)
-Werror,-Wunused-but-set-variable (Clang 13 trunk)
Since 20e5f64215853bdd32c5f16394ba7f2f36745904 "loplugin:unused-returns in sw" removed > return bChanged || bTabChanged; from the end of SwLayAction::FormatLayoutFly, bTabChanged is effectively completely unused (and now gets warned about by Clang), and the update of bChanged is apparently no longer used, either. Change-Id: Iab180e83070c45531aecf46d85a030463cf7b22d Reviewed-on: https://gerrit.libreoffice.org/c/core/+/116853 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sw/source')
-rw-r--r--sw/source/core/layout/layact.cxx5
1 files changed, 2 insertions, 3 deletions
diff --git a/sw/source/core/layout/layact.cxx b/sw/source/core/layout/layact.cxx
index 0ab451c111a0..4e95504f07ef 100644
--- a/sw/source/core/layout/layact.cxx
+++ b/sw/source/core/layout/layact.cxx
@@ -1430,16 +1430,15 @@ void SwLayAction::FormatLayoutFly( SwFlyFrame* pFly )
return;
// Now, deal with the lowers that are LayoutFrames
- bool bTabChanged = false;
SwFrame *pLow = pFly->Lower();
while ( pLow )
{
if ( pLow->IsLayoutFrame() )
{
if ( pLow->IsTabFrame() )
- bTabChanged |= FormatLayoutTab( static_cast<SwTabFrame*>(pLow), bAddRect );
+ FormatLayoutTab( static_cast<SwTabFrame*>(pLow), bAddRect );
else
- bChanged |= FormatLayout( m_pImp->GetShell()->GetOut(), static_cast<SwLayoutFrame*>(pLow), bAddRect );
+ FormatLayout( m_pImp->GetShell()->GetOut(), static_cast<SwLayoutFrame*>(pLow), bAddRect );
}
pLow = pLow->GetNext();
}