diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-06-23 15:02:35 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-06-24 08:43:55 +0200 |
| commit | a2fc883173d7053cefe543620982051ae40c4b03 (patch) | |
| tree | dac1b760925c8151dfd8a9cbe3a8735b68ab9f79 /unotools/source/config | |
| parent | 010713e65ccade7b682c219707c8db3d864145c1 (diff) | |
use more std::container::insert instead of std::copy
which is both more compact code, and more efficient, since the insert
method can do smarter resizing
Change-Id: I17f226660f87cdf002edccc29b4af8fd59a25f91
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/96948
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'unotools/source/config')
| -rw-r--r-- | unotools/source/config/dynamicmenuoptions.cxx | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/unotools/source/config/dynamicmenuoptions.cxx b/unotools/source/config/dynamicmenuoptions.cxx index 019ca1b86407..2421d228fdf0 100644 --- a/unotools/source/config/dynamicmenuoptions.cxx +++ b/unotools/source/config/dynamicmenuoptions.cxx @@ -488,8 +488,7 @@ void SvtDynamicMenuOptions_Impl::impl_SortAndExpandPropertyNames( const Sequence lDestination.realloc( (nSourceCount*PROPERTYCOUNT)+nDestinationStep ); // get enough memory for copy operations after nDestination ... // Copy all items to temp. vector to use fast sort operations :-) - lTemp.reserve(nSourceCount); - std::copy(lSource.begin(), lSource.end(), std::back_inserter(lTemp)); + lTemp.insert( lTemp.end(), lSource.begin(), lSource.end() ); // Sort all entries by number ... stable_sort( lTemp.begin(), lTemp.end(), CountWithPrefixSort() ); |