From 7133be045fcec7db8a9a970b51df2efdc9c18b94 Mon Sep 17 00:00:00 2001 From: Asad Tauqeer Date: Wed, 29 Apr 2026 16:53:09 +0500 Subject: [PATCH] tdf#153294 Use std::find_if in autocdlg.cxx Replaced for loop with std::find_if in OfaAutocorrReplacePage::NewEntry Change-Id: Ib7cef11e0b661883e6875045daa9d57483dd3e08 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/204571 Tested-by: Jenkins Reviewed-by: Julien Nabet --- cui/source/tabpages/autocdlg.cxx | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/cui/source/tabpages/autocdlg.cxx b/cui/source/tabpages/autocdlg.cxx index a7ca119fad63..2c80c4ced232 100644 --- a/cui/source/tabpages/autocdlg.cxx +++ b/cui/source/tabpages/autocdlg.cxx @@ -1000,14 +1000,10 @@ void OfaAutocorrReplacePage::NewEntry(const OUString& sShort, const OUString& sL rNewArray.erase(it); DoubleStringArray& rDeletedArray = aChangesTable[eLang].aDeletedEntries; - for (size_t i = 0; i < rDeletedArray.size(); i++) - { - if (rDeletedArray[i].sShort == sShort) - { - rDeletedArray.erase(rDeletedArray.begin() + i); - break; - } - } + auto itDeleted = std::find_if(rDeletedArray.begin(), rDeletedArray.end(), + [&sShort](const DoubleString& rEntry) { return rEntry.sShort == sShort; }); + if (itDeleted != rDeletedArray.end()) + rDeletedArray.erase(itDeleted); DoubleString aNewString(sShort, sLong); rNewArray.push_back(aNewString);