[Libreoffice-commits] core.git: cppu/source cui/source dbaccess/source

Arnold Dumas arnold at dumas.at
Mon Feb 15 07:38:46 UTC 2016


 cppu/source/uno/lbenv.cxx                              |    5 +----
 cui/source/options/optjava.cxx                         |   11 +++++------
 dbaccess/source/ext/macromigration/migrationengine.cxx |    2 +-
 3 files changed, 7 insertions(+), 11 deletions(-)

New commits:
commit 53bf3ea58e0f4f9f4399a44bfc9ddecc71cb54c7
Author: Arnold Dumas <arnold at dumas.at>
Date:   Sun Feb 14 00:59:35 2016 +0100

    tdf#57950: Replace chained O(U)StringBuffer::append() with operator+
    
    Change-Id: Ic76cf0130ebec4cf723b83d82f59ae45bfcb3ea1
    Reviewed-on: https://gerrit.libreoffice.org/22350
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Noel Grandin <noelgrandin at gmail.com>

diff --git a/cppu/source/uno/lbenv.cxx b/cppu/source/uno/lbenv.cxx
index b9e964da..c5cdaa5 100644
--- a/cppu/source/uno/lbenv.cxx
+++ b/cppu/source/uno/lbenv.cxx
@@ -1108,10 +1108,7 @@ static uno_Environment * initDefaultEnvironment(
     else
     {
         // late init with some code from matching uno language binding
-        ::rtl::OUStringBuffer aLibName( 16 );
-        aLibName.append( envTypeName );
-        aLibName.append( "_uno" );
-        OUString aStr( aLibName.makeStringAndClear() );
+        OUString aStr( envTypeName + "_uno" );
 
         if (!loadEnv(aStr, pEnv))
         {
diff --git a/cui/source/options/optjava.cxx b/cui/source/options/optjava.cxx
index 4bcdeab..3f178ac 100644
--- a/cui/source/options/optjava.cxx
+++ b/cui/source/options/optjava.cxx
@@ -167,12 +167,11 @@ SvxJavaOptionsPage::SvxJavaOptionsPage( vcl::Window* pParent, const SfxItemSet&
 
     m_pJavaList->SvSimpleTable::SetTabs( aStaticTabs );
 
-    OUStringBuffer sHeader;
-    sHeader.append("\t").append(get<FixedText>("vendor")->GetText())
-        .append("\t").append(get<FixedText>("version")->GetText())
-        .append("\t").append(get<FixedText>("features")->GetText())
-        .append("\t");
-    m_pJavaList->InsertHeaderEntry(sHeader.makeStringAndClear(), HEADERBAR_APPEND, HeaderBarItemBits::LEFT);
+    OUString sHeader ( "\t" + get<FixedText>("vendor")->GetText() +
+        "\t" + get<FixedText>("version")->GetText() +
+        "\t" + get<FixedText>("features")->GetText() +
+        "\t" );
+    m_pJavaList->InsertHeaderEntry(sHeader, HEADERBAR_APPEND, HeaderBarItemBits::LEFT);
     m_pJavaList->setColSizes();
 
     m_pJavaEnableCB->SetClickHdl( LINK( this, SvxJavaOptionsPage, EnableHdl_Impl ) );
diff --git a/dbaccess/source/ext/macromigration/migrationengine.cxx b/dbaccess/source/ext/macromigration/migrationengine.cxx
index 2aa941c..94dd12c 100644
--- a/dbaccess/source/ext/macromigration/migrationengine.cxx
+++ b/dbaccess/source/ext/macromigration/migrationengine.cxx
@@ -991,7 +991,7 @@ namespace dbmm
         {
             const OUString sHierarhicalBase(
                 _rContainerLoc.isEmpty() ? OUString() :
-                                           OUStringBuffer( _rContainerLoc ).append( "/" ).makeStringAndClear());
+                                           OUString( _rContainerLoc +  "/" ) );
 
             Sequence< OUString > aElementNames( _rxContainer->getElementNames() );
             for (   const OUString* elementName = aElementNames.getConstArray();


More information about the Libreoffice-commits mailing list