use std::vector::insert instead of push_back

because it will pre-allocate space and often is optimised to memcpy

Change-Id: I03ed7915f2762d3d27e378638052a47a28bbf096
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/123588
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

1 files changed, 2 insertions, 4 deletions

diff --git a/tools/source/memtools/multisel.cxx b/tools/source/memtools/multisel.cxx
index faa78f78144a..1f3f8eb5a23a 100644
--- a/tools/source/memtools/multisel.cxx
+++ b/tools/source/memtools/multisel.cxx
@@ -88,8 +88,7 @@ MultiSelection::MultiSelection( const MultiSelection& rOrig ) :
     }
 
     // copy the sub selections
-    for (const Range & rSel : rOrig.aSels)
-        aSels.push_back( rSel );
+    aSels.insert( aSels.end(), rOrig.aSels.begin(), rOrig.aSels.end() );
 }
 
 MultiSelection::MultiSelection( const Range& rRange ):
@@ -117,8 +116,7 @@ MultiSelection& MultiSelection::operator= ( const MultiSelection& rOrig )
 
     // clear the old and copy the sub selections
     ImplClear();
-    for (const Range& rSel : rOrig.aSels)
-        aSels.push_back( rSel );
+    aSels.insert( aSels.end(), rOrig.aSels.begin(), rOrig.aSels.end() );
     nSelCount = rOrig.nSelCount;
 
     return *this;