chart2/qa/extras/chart2import.cxx     |    2 +-
 sc/qa/unit/scshapetest.cxx            |    6 +++---
 sc/qa/unit/subsequent_export-test.cxx |    2 +-
 sc/source/filter/excel/xeextlst.cxx   |   10 +++++++---
 4 files changed, 12 insertions(+), 8 deletions(-)

New commits:
commit 84b08eaeb7b2e8a61ee0dab79cabae07e2209c3f
Author:     Stephan Bergmann <sberg...@redhat.com>
AuthorDate: Tue Dec 29 12:19:45 2020 +0100
Commit:     Stephan Bergmann <sberg...@redhat.com>
CommitDate: Wed Dec 30 11:06:20 2020 +0100

    loplugin:stringviewparam: operator +, redux
    
    Change-Id: I0790e09f24512d7b205681127eb95ad7dc15a15f
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108473
    Reviewed-by: Mike Kaganski <mike.kagan...@collabora.com>
    Tested-by: Stephan Bergmann <sberg...@redhat.com>

diff --git a/chart2/qa/extras/chart2import.cxx 
b/chart2/qa/extras/chart2import.cxx
index 272639ad069c..57c7227b49c1 100644
--- a/chart2/qa/extras/chart2import.cxx
+++ b/chart2/qa/extras/chart2import.cxx
@@ -2806,7 +2806,7 @@ void Chart2ImportTest::testTdfCustomShapePos()
 
 void Chart2ImportTest::testTdf121281()
 {
-    load("/chart2/qa/extras/data/xlsx/", "incorrect_label_position.xlsx");
+    load(u"/chart2/qa/extras/data/xlsx/", "incorrect_label_position.xlsx");
     Reference<chart::XChartDocument> xChartDoc(getChartDocFromSheet(0, 
mxComponent),
                                                UNO_QUERY_THROW);
     Reference<drawing::XDrawPageSupplier> xDrawPageSupplier(xChartDoc, 
UNO_QUERY_THROW);
diff --git a/sc/qa/unit/scshapetest.cxx b/sc/qa/unit/scshapetest.cxx
index d9fa04683ea6..727997a8c8c0 100644
--- a/sc/qa/unit/scshapetest.cxx
+++ b/sc/qa/unit/scshapetest.cxx
@@ -144,7 +144,7 @@ void ScShapeTest::testTdf137033_FlipHori_Resize()
     // Load a document, which has a rotated custom shape, which is horizontal 
flipped. Error was, that
     // if such shape was anchored "resize with cell", then after save and 
reload it was destorted.
     OUString aFileURL;
-    createFileURL("tdf137033_FlipHoriRotCustomShape.ods", aFileURL);
+    createFileURL(u"tdf137033_FlipHoriRotCustomShape.ods", aFileURL);
     uno::Reference<css::lang::XComponent> xComponent = 
loadFromDesktop(aFileURL);
     CPPUNIT_ASSERT(xComponent.is());
 
@@ -203,7 +203,7 @@ void ScShapeTest::testTdf137033_RotShear_ResizeHide()
 
     // Load a document, which has a rotated and sheared shape, anchored to 
cell with resize.
     OUString aFileURL;
-    createFileURL("tdf137033_RotShearResizeAnchor.ods", aFileURL);
+    createFileURL(u"tdf137033_RotShearResizeAnchor.ods", aFileURL);
     uno::Reference<css::lang::XComponent> xComponent = 
loadFromDesktop(aFileURL);
     CPPUNIT_ASSERT(xComponent.is());
 
@@ -278,7 +278,7 @@ void ScShapeTest::testTdf137033_RotShear_Hide()
 
     // Load a document, which has a rotated and sheared shape, anchored to 
cell, without resize.
     OUString aFileURL;
-    createFileURL("tdf137033_RotShearCellAnchor.ods", aFileURL);
+    createFileURL(u"tdf137033_RotShearCellAnchor.ods", aFileURL);
     uno::Reference<css::lang::XComponent> xComponent = 
loadFromDesktop(aFileURL);
     CPPUNIT_ASSERT(xComponent.is());
 
diff --git a/sc/qa/unit/subsequent_export-test.cxx 
b/sc/qa/unit/subsequent_export-test.cxx
index f93f24b2688a..0261ea9f2dca 100644
--- a/sc/qa/unit/subsequent_export-test.cxx
+++ b/sc/qa/unit/subsequent_export-test.cxx
@@ -521,7 +521,7 @@ void ScExportTest::test()
 
 void ScExportTest::testExtCondFormatXLSX()
 {
-    ScDocShellRef xShell = loadDoc("tdf139021.", FORMAT_XLSX);
+    ScDocShellRef xShell = loadDoc(u"tdf139021.", FORMAT_XLSX);
     CPPUNIT_ASSERT(xShell.is());
 
     ScDocShellRef xDocSh = saveAndReload(&(*xShell), FORMAT_XLSX);
diff --git a/sc/source/filter/excel/xeextlst.cxx 
b/sc/source/filter/excel/xeextlst.cxx
index 1f4055aff5ea..972a9010b342 100644
--- a/sc/source/filter/excel/xeextlst.cxx
+++ b/sc/source/filter/excel/xeextlst.cxx
@@ -7,6 +7,10 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/.
  */
 
+#include <sal/config.h>
+
+#include <string_view>
+
 #include <xeextlst.hxx>
 #include <xeroot.hxx>
 #include <xestyle.hxx>
@@ -164,7 +168,7 @@ bool RequiresFixedFormula(ScConditionMode eMode)
     return false;
 }
 
-OString GetFixedFormula(ScConditionMode eMode, const ScAddress& rAddress, 
const OString& rText)
+OString GetFixedFormula(ScConditionMode eMode, const ScAddress& rAddress, 
std::string_view rText)
 {
     OStringBuffer aBuffer;
     XclXmlUtils::ToOString(aBuffer, rAddress);
@@ -176,9 +180,9 @@ OString GetFixedFormula(ScConditionMode eMode, const 
ScAddress& rAddress, const
     case ScConditionMode::EndsWith:
         return OString("RIGHT(" + aPos + ",LEN(" + rText + "))=\"" + rText + 
"\"");
     case ScConditionMode::ContainsText:
-        return OString("NOT(ISERROR(SEARCH(" + rText + "," + aPos + ")))");
+        return OString(OString::Concat("NOT(ISERROR(SEARCH(") + rText + "," + 
aPos + ")))");
     case ScConditionMode::NotContainsText:
-        return OString("ISERROR(SEARCH(" + rText + "," + aPos + "))");
+        return OString(OString::Concat("ISERROR(SEARCH(") + rText + "," + aPos 
+ "))");
     default:
         break;
     }
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to