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

[Adam Seskunas (via logerrit)]

 filter/source/msfilter/escherex.cxx |    3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

New commits:
commit d039fcd1cc6ce1f746b412f364790614bf972740
Author:     Adam Seskunas <adamsesku...@gmail.com>
AuthorDate: Thu Sep 14 21:02:14 2023 -0700
Commit:     Mike Kaganski <mike.kagan...@collabora.com>
CommitDate: Fri Sep 15 07:28:13 2023 +0200

    Move nUnicode variable to for loop condition statement.
    
    Use nUnicode directly in range based loop condition statement
    instead of declaring seperately in loop body.
    
    Change-Id: I07d8fb7a693909e0cf43ae0562547c51d001eb14
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156936
    Tested-by: Jenkins
    Reviewed-by: Mike Kaganski <mike.kagan...@collabora.com>

diff --git a/filter/source/msfilter/escherex.cxx 
b/filter/source/msfilter/escherex.cxx
index 923bd84da756..19cdd2f3187a 100644
--- a/filter/source/msfilter/escherex.cxx
+++ b/filter/source/msfilter/escherex.cxx
@@ -214,9 +214,8 @@ void EscherPropertyContainer::AddOpt(
     std::vector<sal_uInt8> aBuf;
     aBuf.reserve(rString.size() * 2 + 2);
 
-    for(const auto& nChar: rString)
+    for(const sal_Unicode nUnicode: rString)
     {
-        const sal_Unicode nUnicode(nChar);
         aBuf.push_back(static_cast<sal_uInt8>(nUnicode));
         aBuf.push_back(static_cast<sal_uInt8>(nUnicode >> 8));
     }