loplugin:stringloop in svgio..xmlsecurity

Change-Id: I1b2fe5674c8350690efc3d3219b9273cc61d5b0c
Reviewed-on: https://gerrit.libreoffice.org/58332
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

1 files changed, 4 insertions, 5 deletions

diff --git a/uui/source/secmacrowarnings.cxx b/uui/source/secmacrowarnings.cxx
index 1233ced8c86c..3030ab92520a 100644
--- a/uui/source/secmacrowarnings.cxx
+++ b/uui/source/secmacrowarnings.cxx
@@ -160,16 +160,15 @@ void MacroWarning::SetStorage( const css::uno::Reference < css::embed::XStorage
     {
         mpInfos = &rInfos;
         OUString aCN_Id("CN");
-        OUString s;
-        s = GetContentPart( rInfos[ 0 ].Signer->getSubjectName(), aCN_Id );
+        OUStringBuffer s = GetContentPart( rInfos[ 0 ].Signer->getSubjectName(), aCN_Id );
 
         for( sal_Int32 i = 1 ; i < nCnt ; ++i )
         {
-            s += "\n";
-            s += GetContentPart( rInfos[ i ].Signer->getSubjectName(), aCN_Id );
+            s.append("\n");
+            s.append(GetContentPart( rInfos[ i ].Signer->getSubjectName(), aCN_Id ));
         }
 
-        mxSignsFI->set_label(s);
+        mxSignsFI->set_label(s.makeStringAndClear());
         mxViewSignsBtn->set_sensitive(true);
     }
 }