[Libreoffice-commits] core.git: cui/source

[Stephan Bergmann (via logerrit)]

 cui/source/options/fontsubs.cxx |   11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

New commits:
commit 6dbccf3d9344be0c2ba162a8df5fc0bf425cb84e
Author:     Stephan Bergmann <sberg...@redhat.com>
AuthorDate: Wed Jun 1 08:46:33 2022 +0200
Commit:     Stephan Bergmann <sberg...@redhat.com>
CommitDate: Wed Jun 1 09:54:55 2022 +0200

    Clean up some loop a bit
    
    Change-Id: I96b760b7bd220b799f4843c5d5e07385a9a995d2
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135215
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <sberg...@redhat.com>

diff --git a/cui/source/options/fontsubs.cxx b/cui/source/options/fontsubs.cxx
index b5c98b76663b..93c8b03d98b3 100644
--- a/cui/source/options/fontsubs.cxx
+++ b/cui/source/options/fontsubs.cxx
@@ -204,14 +204,13 @@ void  SvxFontSubstTabPage::Reset( const SfxItemSet* )
 
     std::vector<SubstitutionStruct> aFontSubs = 
svtools::GetFontSubstitutions();
     std::unique_ptr<weld::TreeIter> xIter(m_xCheckLB->make_iterator());
-    for (auto const & i: aFontSubs)
+    for (auto const & sub: aFontSubs)
     {
         m_xCheckLB->append(xIter.get());
-        const SubstitutionStruct* pSubs = &i;
-        m_xCheckLB->set_toggle(*xIter, pSubs->bReplaceAlways ? TRISTATE_TRUE : 
TRISTATE_FALSE, 0);
-        m_xCheckLB->set_toggle(*xIter, pSubs->bReplaceOnScreenOnly ? 
TRISTATE_TRUE : TRISTATE_FALSE, 1);
-        m_xCheckLB->set_text(*xIter, pSubs->sFont, 2);
-        m_xCheckLB->set_text(*xIter, pSubs->sReplaceBy, 3);
+        m_xCheckLB->set_toggle(*xIter, sub.bReplaceAlways ? TRISTATE_TRUE : 
TRISTATE_FALSE, 0);
+        m_xCheckLB->set_toggle(*xIter, sub.bReplaceOnScreenOnly ? 
TRISTATE_TRUE : TRISTATE_FALSE, 1);
+        m_xCheckLB->set_text(*xIter, sub.sFont, 2);
+        m_xCheckLB->set_text(*xIter, sub.sReplaceBy, 3);
     }
 
     m_xCheckLB->thaw();