sw/source/ui/config/optload.cxx | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
New commits:
commit 0ce0f5ee14e377ea8f7b52b4d7228ead1089f7c5
Author: Simon Chenery <simon_chen...@yahoo.com>
AuthorDate: Sun Jul 6 15:53:55 2025 +0200
Commit: David Gilbert <freedesk...@treblig.org>
CommitDate: Wed Jul 9 23:03:13 2025 +0200
tdf#145538 Use range based for loop in optload.cxx
Change-Id: I64b3fa8ce4c962b10fc87cb50ae548e5e4dc07cd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/187446
Reviewed-by: David Gilbert <freedesk...@treblig.org>
Reviewed-by: Mike Kaganski <mike.kagan...@collabora.com>
Tested-by: Jenkins
diff --git a/sw/source/ui/config/optload.cxx b/sw/source/ui/config/optload.cxx
index 0d5e2d727297..618673ddde3c 100644
--- a/sw/source/ui/config/optload.cxx
+++ b/sw/source/ui/config/optload.cxx
@@ -535,11 +535,8 @@ SwLoadOptPage::SwLoadOptPage(weld::Container* pPage,
weld::DialogController* pCo
,
m_xStandardizedPageSizeNF(m_xBuilder->weld_spin_button(u"standardpagesize"_ustr))
,
m_xStandardizedPageSizeImg(m_xBuilder->weld_widget(u"lockstandardpagesize"_ustr))
{
- for (sal_uInt32 i = 0; i < SwFieldUnitTable::Count(); ++i)
+ for (const auto& [pId, eFUnit] : STR_ARR_METRIC)
{
- const OUString sMetric = SwFieldUnitTable::GetString(i);
- FieldUnit eFUnit = SwFieldUnitTable::GetValue(i);
-
switch ( eFUnit )
{
case FieldUnit::MM:
@@ -549,7 +546,7 @@ SwLoadOptPage::SwLoadOptPage(weld::Container* pPage,
weld::DialogController* pCo
case FieldUnit::INCH:
{
// use only these metrics
-
m_xMetricLB->append(OUString::number(static_cast<sal_uInt32>(eFUnit)), sMetric);
+
m_xMetricLB->append(OUString::number(static_cast<sal_uInt32>(eFUnit)),
SwResId(pId));
break;
}
default:; //prevent warning
