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

ekuiitr jhaekansh80 at gmail.com
Sat Oct 14 21:56:40 UTC 2017


 sw/source/ui/vba/vbacolumn.cxx |    8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

New commits:
commit 8eacd3be08bf6e1a97900624611822de9b00a379
Author: ekuiitr <jhaekansh80 at gmail.com>
Date:   Sun Oct 15 02:03:15 2017 +0530

    tdf#112689 - Replace chained O(U)StringBuffer::append() with operator+
    
    changed OUStringBuffer::append() to op+. deleted unnecessary variable.
    
    Change-Id: Ibe27f7f587f7130e7c31e33a29a88c5ea6478740
    Reviewed-on: https://gerrit.libreoffice.org/43395
    Reviewed-by: Julien Nabet <serval2412 at yahoo.fr>
    Tested-by: Jenkins <ci at libreoffice.org>

diff --git a/sw/source/ui/vba/vbacolumn.cxx b/sw/source/ui/vba/vbacolumn.cxx
index 89ef096d97c1..676d93e1712f 100644
--- a/sw/source/ui/vba/vbacolumn.cxx
+++ b/sw/source/ui/vba/vbacolumn.cxx
@@ -62,16 +62,14 @@ SwVbaColumn::Select( )
 
 void SwVbaColumn::SelectColumn( const uno::Reference< frame::XModel >& xModel, const uno::Reference< text::XTextTable >& xTextTable, sal_Int32 nStartColumn, sal_Int32 nEndColumn )
 {
-    OUStringBuffer aRangeName;
     OUString sStartCol = SwVbaTableHelper::getColumnStr( nStartColumn );
-    aRangeName.append(sStartCol).append(sal_Int32( 1 ) );
+    OUString aRangeName = sStartCol + OUString::number( 1 );
     OUString sEndCol = SwVbaTableHelper::getColumnStr( nEndColumn );
     sal_Int32 nRowCount = xTextTable->getRows()->getCount();
-    aRangeName.append(':').append( sEndCol ).append( nRowCount );
+    aRangeName += ":" + sEndCol + OUString::number(nRowCount);
 
     uno::Reference< table::XCellRange > xCellRange( xTextTable, uno::UNO_QUERY_THROW );
-    OUString sSelRange = aRangeName.makeStringAndClear();
-    uno::Reference< table::XCellRange > xSelRange = xCellRange->getCellRangeByName( sSelRange );
+    uno::Reference< table::XCellRange > xSelRange = xCellRange->getCellRangeByName( aRangeName );
 
     uno::Reference< view::XSelectionSupplier > xSelection( xModel->getCurrentController(), uno::UNO_QUERY_THROW );
     xSelection->select( uno::makeAny( xSelRange ) );


More information about the Libreoffice-commits mailing list