sw/source/core/layout/tabfrm.cxx | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
New commits: commit 669a69f5e5590585c6f8e44304e26a9b0f0730d2 Author: Mike Kaganski <[email protected]> AuthorDate: Mon Jan 26 07:01:28 2026 +0100 Commit: Mike Kaganski <[email protected]> CommitDate: Mon Jan 26 08:12:55 2026 +0100 Move variable into its scope Any variable in too broad scope may cause headaches; but for methods like SwTabFrame::MakeAll (more than a thousand lines), it easily hits with "the variable hides earlier one" errors. Change-Id: I6b8e2137b777698a4f82d1032ceac6daed6a7eba Reviewed-on: https://gerrit.libreoffice.org/c/core/+/198117 Tested-by: Jenkins Reviewed-by: Mike Kaganski <[email protected]> diff --git a/sw/source/core/layout/tabfrm.cxx b/sw/source/core/layout/tabfrm.cxx index 6b71fc26c91b..f37ea940b18e 100644 --- a/sw/source/core/layout/tabfrm.cxx +++ b/sw/source/core/layout/tabfrm.cxx @@ -2927,8 +2927,7 @@ void SwTabFrame::MakeAll(vcl::RenderContext* pRenderContext) // 3. The table is allowed to split or we do not have a pIndPrev: SwFrame* pIndPrev = GetIndPrev(); - SwFlyFrame* pFly = FindFlyFrame(); - if (!pIndPrev && pFly && pFly->IsFlySplitAllowed()) + if (SwFlyFrame* pFly = FindFlyFrame(); !pIndPrev && pFly && pFly->IsFlySplitAllowed()) { if (SwFrame* pAnchor = pFly->FindAnchorCharFrame()) {
