sc/source/ui/Accessibility/AccessibleCell.cxx |   23 +++++------------------
 1 file changed, 5 insertions(+), 18 deletions(-)

New commits:
commit 884088de799c2c6947b72eb38cf2e7dbc46a1867
Author:     Julien Nabet <serval2...@yahoo.fr>
AuthorDate: Sun Sep 4 10:14:03 2022 +0200
Commit:     Julien Nabet <serval2...@yahoo.fr>
CommitDate: Sun Sep 4 15:25:54 2022 +0200

    Simplify by using replaceAll in a loop (sc/AccessibleCell.cxx)
    
    Change-Id: I320f9631fb7c8c6d0c34fde1ad6c7b9479aa57db
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/139357
    Tested-by: Jenkins
    Reviewed-by: Julien Nabet <serval2...@yahoo.fr>

diff --git a/sc/source/ui/Accessibility/AccessibleCell.cxx 
b/sc/source/ui/Accessibility/AccessibleCell.cxx
index 854ad84f961d..96020a08ef63 100644
--- a/sc/source/ui/Accessibility/AccessibleCell.cxx
+++ b/sc/source/ui/Accessibility/AccessibleCell.cxx
@@ -473,26 +473,13 @@ void ScAccessibleCell::AddRelation(const ScRange& rRange,
     pRelationSet->AddRelation(aRelation);
 }
 
-static OUString ReplaceOneChar(const OUString& oldOUString, 
std::u16string_view replacedChar, std::u16string_view replaceStr)
-{
-    int iReplace = oldOUString.lastIndexOf(replacedChar);
-    OUString aRet = oldOUString;
-    while(iReplace > -1)
-    {
-        aRet = aRet.replaceAt(iReplace, 1, replaceStr);
-        iReplace = aRet.lastIndexOf(replacedChar, iReplace);
-    }
-    return aRet;
-}
-
 static OUString ReplaceFourChar(const OUString& oldOUString)
 {
-    OUString aRet = ReplaceOneChar(oldOUString, u"\\", u"\\\\");
-    aRet = ReplaceOneChar(aRet, u";", u"\\;");
-    aRet = ReplaceOneChar(aRet, u"=", u"\\=");
-    aRet = ReplaceOneChar(aRet, u",", u"\\,");
-    aRet = ReplaceOneChar(aRet, u":", u"\\:");
-    return aRet;
+    return oldOUString.replaceAll(u"\\", u"\\\\")
+        .replaceAll(u";", u"\\;")
+        .replaceAll(u"=", u"\\=")
+        .replaceAll(u",", u"\\,")
+        .replaceAll(u":", u"\\:");
 }
 
 uno::Any SAL_CALL ScAccessibleCell::getExtendedAttributes()

Reply via email to