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

Noel Grandin (via logerrit) logerrit at kemper.freedesktop.org
Mon May 3 10:34:27 UTC 2021


 sc/source/ui/dbgui/asciiopt.cxx |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 165b7b76e6ca1d66d765b4910b2be75384845f41
Author:     Noel Grandin <noel.grandin at collabora.co.uk>
AuthorDate: Mon May 3 09:31:35 2021 +0200
Commit:     Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Mon May 3 12:33:47 2021 +0200

    tdf#142040 CSV import broken in master
    
    regression from
        commit 75997f13ee3a71d6c994392264b0190bd7bb6756
    no need to create temporaries when appending number to
    O[U]StringBuffer
    
    Change-Id: Iacd74475ef7e9a70f60cedf80d3dfe4f1350f40c
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/115024
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>

diff --git a/sc/source/ui/dbgui/asciiopt.cxx b/sc/source/ui/dbgui/asciiopt.cxx
index 0d584b5291a6..270ca6b01069 100644
--- a/sc/source/ui/dbgui/asciiopt.cxx
+++ b/sc/source/ui/dbgui/asciiopt.cxx
@@ -192,7 +192,7 @@ OUString ScAsciiOptions::WriteToString() const
         {
             if (i)
                 aOutStr.append("/");
-            aOutStr.append(aFieldSeps[i]);
+            aOutStr.append(OUString::number(aFieldSeps[i]));
         }
         if ( bMergeFieldSeps )
         {
@@ -202,7 +202,7 @@ OUString ScAsciiOptions::WriteToString() const
     }
 
     // Token 1: Text Quote character.
-    aOutStr.append(",").append(cTextSep).append(",");
+    aOutStr.append("," + OUString::number(cTextSep) + ",");
 
     //Token 2: Text encoding.
     if ( bCharSetSystem )           // force "SYSTEM"


More information about the Libreoffice-commits mailing list