fdo#57950: Remove some chained appends in filter in sw/access

Change-Id: I9f3b44d6ea154d628dd59e31d9403097809a2853
Reviewed-on: https://gerrit.libreoffice.org/2300Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>

sw/source/core/access/accmap.cxx

@@ -175,9 +175,7 @@ void SwDrawModellListener_Impl::Notify( SfxBroadcaster& /*rBC*/,
         {
             (void)r;
 #if OSL_DEBUG_LEVEL > 0
-            rtl::OStringBuffer aError(RTL_CONSTASCII_STRINGPARAM("Runtime exception caught while notifying shape.:\n"));
-            aError.append(rtl::OUStringToOString(r.Message, RTL_TEXTENCODING_ASCII_US));
-            OSL_FAIL( aError.getStr() );
+              OSL_FAIL( "Runtime exception caught while notifying shape.:\n" + OUStringToOString(r.Message, RTL_TEXTENCODING_ASCII_US) );
 #endif
         }
     }

sw/source/core/access/accportions.cxx

@@ -141,40 +141,34 @@ void SwAccessiblePortionData::Special(
 
     // construct string with representation; either directly from
     // rText, or use resources for special case portions
-    String sDisplay;
+    OUString sDisplay;
     switch( nType )
     {
         case POR_POSTITS:
         case POR_FLYCNT:
         case POR_GRFNUM:
-            sDisplay = rtl::OUString(sal_Unicode(0xfffc));
+            sDisplay = OUString(sal_Unicode(0xfffc));
 
             break;
         case POR_NUMBER:
         {
-            OUStringBuffer aTmpBuffer( rText.Len() + 1 );
-            aTmpBuffer.append( rText );
-            aTmpBuffer.append( sal_Unicode(' ') );
-            sDisplay = aTmpBuffer.makeStringAndClear();
+            sDisplay = OUString( rText ) + " ";
             break;
         }
         // #i111768# - apply patch from kstribley:
         // Include the control characters.
         case POR_CONTROLCHAR:
         {
-            OUStringBuffer aTmpBuffer( rText.Len() + 1 );
-            aTmpBuffer.append( rText );
-            aTmpBuffer.append( pTxtNode->GetTxt()[nModelPosition] );
-            sDisplay = aTmpBuffer.makeStringAndClear();
+            sDisplay = OUString( rText ) + OUString( pTxtNode->GetTxt()[nModelPosition] );
             break;
         }
         default:
-            sDisplay = rText;
+            sDisplay = OUString( rText );
             break;
     }
 
     // ignore zero/zero portions (except for terminators)
-    if( (nLength == 0) && (sDisplay.Len() == 0) && (nType != POR_TERMINATE) )
+    if( (nLength == 0) && (sDisplay.getLength() == 0) && (nType != POR_TERMINATE) )
         return;
 
     // special treatment for zero length portion at the beginning:
@@ -194,7 +188,7 @@ void SwAccessiblePortionData::Special(
     aPortionAttrs.push_back( nAttr );
 
     // update buffer + nModelPosition
-    aBuffer.append( OUString(sDisplay) );
+    aBuffer.append( sDisplay );
     nModelPosition += nLength;
 
     // remember 'last' special portion (unless it's our own 'closing'

sw/source/core/access/acctable.cxx

@@ -692,12 +692,7 @@ SwAccessibleTable::SwAccessibleTable(
     const_cast< SwFrmFmt * >( pFrmFmt )->Add( this );
     const String& rName = pFrmFmt->GetName();
 
-    OUStringBuffer aBuffer( rName.Len() + 4 );
-    aBuffer.append( OUString(rName) );
-    aBuffer.append( static_cast<sal_Unicode>( '-' ) );
-    aBuffer.append( static_cast<sal_Int32>( pTabFrm->GetPhyPageNum() ) );
-
-    SetName( aBuffer.makeStringAndClear() );
+    SetName( OUString( rName ) + "-" + OUString::number( pTabFrm->GetPhyPageNum() ) );
 
     OUString sArg1( static_cast< const SwTabFrm * >( GetFrm() )
                                         ->GetFmt()->GetName() );
@@ -728,12 +723,9 @@ void SwAccessibleTable::Modify( const SfxPoolItem* pOld, const SfxPoolItem *pNew
             OUString sOldName( GetName() );
 
             const String& rNewTabName = pFrmFmt->GetName();
-            OUStringBuffer aBuffer( rNewTabName.Len() + 4 );
-            aBuffer.append( OUString(rNewTabName) );
-            aBuffer.append( static_cast<sal_Unicode>( '-' ) );
-            aBuffer.append( static_cast<sal_Int32>( pTabFrm->GetPhyPageNum() ) );
 
-            SetName( aBuffer.makeStringAndClear() );
+            SetName( OUString(rNewTabName) + "-" + OUString::number( pTabFrm->GetPhyPageNum() ) );
+
             if( sOldName != GetName() )
             {
                 AccessibleEventObject aEvent;
@@ -1611,17 +1603,9 @@ SwAccessibleTableColHeaders::SwAccessibleTableColHeaders( SwAccessibleMap *pMap2
     const_cast< SwFrmFmt * >( pFrmFmt )->Add( this );
     const String& rName = pFrmFmt->GetName();
 
-    OUStringBuffer aBuffer( rName.Len() + 15 + 6 );
-    aBuffer.append( OUString(rName) );
-    aBuffer.append( rtl::OUString("-ColumnHeaders-") );
-    aBuffer.append( static_cast<sal_Int32>( pTabFrm->GetPhyPageNum() ) );
-
-    SetName( aBuffer.makeStringAndClear() );
+    SetName( OUString(rName) + "-ColumnHeaders-" +  OUString::number( pTabFrm->GetPhyPageNum() ) );
 
-    OUStringBuffer aBuffer2( rName.Len() + 14 );
-    aBuffer2.append( OUString(rName) );
-    aBuffer2.append( rtl::OUString("-ColumnHeaders") );
-    OUString sArg1( aBuffer2.makeStringAndClear() );
+    OUString sArg1( OUString(rName) + "-ColumnHeaders" );
     OUString sArg2( GetFormattedPageNumber() );
 
     OUString sDesc2 = GetResource( STR_ACCESS_TABLE_DESC, &sArg1, &sArg2 );