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

nadith nadmalinda at gmail.com
Fri Jul 15 08:38:24 UTC 2016


 UnoControls/source/controls/progressmonitor.cxx |   12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

New commits:
commit bfe6c2dedbac4caa8a080a7a7b2d783ae5d5d813
Author: nadith <nadmalinda at gmail.com>
Date:   Thu Jul 14 23:35:45 2016 +0530

    tdf#100726: Improve readability of OUString concatenation a bit
    
    Change-Id: I6c51b9004089721f199bc35b1e5c13df36847f02
    Reviewed-on: https://gerrit.libreoffice.org/27227
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Noel Grandin <noelgrandin at gmail.com>

diff --git a/UnoControls/source/controls/progressmonitor.cxx b/UnoControls/source/controls/progressmonitor.cxx
index 62446f5..635e0ea 100644
--- a/UnoControls/source/controls/progressmonitor.cxx
+++ b/UnoControls/source/controls/progressmonitor.cxx
@@ -743,8 +743,7 @@ void ProgressMonitor::impl_rebuildFixedText ()
         // "\n" MUST BE at the end of line!!! => Else ... topic and his text are not in the same line!!!
         for (IMPL_TextlistItem* pSearchItem : maTextlist_Top)
         {
-            aCollectString  +=  pSearchItem->sTopic;
-            aCollectString  +=  "\n";
+            aCollectString  +=  pSearchItem->sTopic + "\n";
         }
 
         m_xTopic_Top->setText ( aCollectString );
@@ -759,8 +758,7 @@ void ProgressMonitor::impl_rebuildFixedText ()
         // "\n" MUST BE at the end of line!!! => Else ... topic and his text are not in the same line!!!
         for (IMPL_TextlistItem* pSearchItem : maTextlist_Top)
         {
-            aCollectString  +=  pSearchItem->sText;
-            aCollectString  +=  "\n";
+            aCollectString  +=  pSearchItem->sText + "\n";
         }
 
         m_xText_Top->setText ( aCollectString );
@@ -777,8 +775,7 @@ void ProgressMonitor::impl_rebuildFixedText ()
         // "\n" MUST BE at the end of line!!! => Else ... topic and his text are not in the same line!!!
         for (IMPL_TextlistItem* pSearchItem : maTextlist_Bottom)
         {
-            aCollectString  +=  pSearchItem->sTopic;
-            aCollectString  +=  "\n";
+            aCollectString  +=  pSearchItem->sTopic + "\n";
         }
 
         m_xTopic_Bottom->setText ( aCollectString );
@@ -793,8 +790,7 @@ void ProgressMonitor::impl_rebuildFixedText ()
         // "\n" MUST BE at the end of line!!! => Else ... topic and his text are not in the same line!!!
         for (IMPL_TextlistItem* pSearchItem : maTextlist_Bottom)
         {
-            aCollectString  +=  pSearchItem->sText;
-            aCollectString  +=  "\n";
+            aCollectString  +=  pSearchItem->sText + "\n";
         }
 
         m_xText_Bottom->setText ( aCollectString );


More information about the Libreoffice-commits mailing list