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

Matteo Casalin matteo.casalin at yahoo.com
Sun Apr 21 10:07:10 PDT 2013


 editeng/source/editeng/textconv.cxx |    7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

New commits:
commit 8e73c8ebcb4110e09d386927ffbe95a49ac5565c
Author: Matteo Casalin <matteo.casalin at yahoo.com>
Date:   Sun Apr 21 19:05:34 2013 +0200

    Avoid multiple appends on OUString
    
    Change-Id: I877c4520be335dc04bbfdfa9a15dab52dd7f446a

diff --git a/editeng/source/editeng/textconv.cxx b/editeng/source/editeng/textconv.cxx
index 0fc00a4..9733842 100644
--- a/editeng/source/editeng/textconv.cxx
+++ b/editeng/source/editeng/textconv.cxx
@@ -349,9 +349,6 @@ void TextConvWrapper::ReplaceUnit(
     if (!bOK)
         return;
 
-    static OUString aBracketedStart( "(" );
-    static OUString aBracketedEnd( ")" );
-
     // select current unit
     SelectNewUnit_impl( nUnitStart, nUnitEnd );
 
@@ -362,10 +359,10 @@ void TextConvWrapper::ReplaceUnit(
         case eExchange :
         break;
         case eReplacementBracketed :
-            (((aNewTxt = aOrigTxt) += aBracketedStart) += rReplaceWith) += aBracketedEnd;
+            aNewTxt = aOrigTxt + "(" + rReplaceWith + ")";
         break;
         case eOriginalBracketed :
-            (((aNewTxt = rReplaceWith) += aBracketedStart) += aOrigTxt) += aBracketedEnd;
+            aNewTxt = rReplaceWith + "(" + aOrigTxt + ")";
         break;
         case eReplacementAbove  :
         case eOriginalAbove :


More information about the Libreoffice-commits mailing list