writerperfect/source/common/WPFTEncodingDialog.cxx |    6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

New commits:
commit f90d00a43f9fd1e14ac42f43c11e5f7eb966a355
Author:     Simon Chenery <simon_chen...@yahoo.com>
AuthorDate: Sun Dec 8 13:14:45 2024 +0100
Commit:     Ilmari Lauhakangas <ilmari.lauhakan...@libreoffice.org>
CommitDate: Wed Dec 25 11:24:13 2024 +0100

    tdf#147021 avoid use of SAL_N_ELEMENTS macro
    
    Change-Id: I320f09b918c75fe83ea7edea2d500a6b2b86b519
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/178081
    Tested-by: Jenkins
    Reviewed-by: Ilmari Lauhakangas <ilmari.lauhakan...@libreoffice.org>

diff --git a/writerperfect/source/common/WPFTEncodingDialog.cxx 
b/writerperfect/source/common/WPFTEncodingDialog.cxx
index dc440929dcea..e0a4f8490582 100644
--- a/writerperfect/source/common/WPFTEncodingDialog.cxx
+++ b/writerperfect/source/common/WPFTEncodingDialog.cxx
@@ -79,12 +79,10 @@ std::pair<std::u16string_view, std::u16string_view> const 
s_encodings[]
         { u"CP865", u"Western Europe (DOS/OS2-865/Nordic)" },
         { u"CP1252", u"Western Europe (Windows-1252/WinLatin 1)" } };
 
-std::size_t const numEncodings = SAL_N_ELEMENTS(s_encodings);
-
 void insertEncodings(weld::ComboBox& box)
 {
-    for (std::size_t i = 0; i < numEncodings; ++i)
-        box.append(OUString(s_encodings[i].first), 
OUString(s_encodings[i].second));
+    for (const auto& rEncoding : s_encodings)
+        box.append(OUString(rEncoding.first), OUString(rEncoding.second));
 }
 
 void selectEncoding(weld::ComboBox& box, const OUString& encoding) { 
box.set_active_id(encoding); }

Reply via email to