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>

1 files changed, 1 insertions, 2 deletions

diff --git a/slideshow/source/engine/slide/layermanager.cxx b/slideshow/source/engine/slide/layermanager.cxx
index 2ec37ee2c6ff..a69f42d31372 100644
--- a/slideshow/source/engine/slide/layermanager.cxx
+++ b/slideshow/source/engine/slide/layermanager.cxx
@@ -722,8 +722,7 @@ namespace slideshow::internal
 
             // to avoid tons of temporaries, create weak_ptr to Layers
             // beforehand
-            std::vector< LayerWeakPtr > aWeakLayers(maLayers.size());
-            std::copy(maLayers.begin(),maLayers.end(),aWeakLayers.begin());
+            std::vector< LayerWeakPtr > aWeakLayers(maLayers.begin(),maLayers.end());
 
             std::size_t                   nCurrLayerIndex(0);
             bool                          bIsBackgroundLayer(true);