[Libreoffice-commits] core.git: basegfx/source include/basegfx slideshow/source

[Noel Grandin (via logerrit)]

 basegfx/source/tools/keystoplerp.cxx                                 |    4 ++--
 include/basegfx/utils/keystoplerp.hxx                                |    2 +-
 slideshow/source/engine/activities/continuouskeytimeactivitybase.cxx |    2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

New commits:
commit e0117eff7f16277e7d6df18be60a6d6138b8611c
Author:     Noel Grandin <noel.grandin at collabora.co.uk>
AuthorDate: Wed Oct 6 15:53:21 2021 +0200
Commit:     Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Wed Oct 6 19:37:44 2021 +0200

    loplugin:moveparam in basegfx
    
    Change-Id: I68d0de099641ae5b2ae92f46e86bdf4a43c468a4
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/123141
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>

diff --git a/basegfx/source/tools/keystoplerp.cxx b/basegfx/source/tools/keystoplerp.cxx
index 0610dd19c228..e5d0d76304e2 100644
--- a/basegfx/source/tools/keystoplerp.cxx
+++ b/basegfx/source/tools/keystoplerp.cxx
@@ -42,8 +42,8 @@ static void validateInput(const std::vector<double>& rKeyStops)
 
 namespace basegfx::utils
 {
-        KeyStopLerp::KeyStopLerp( const std::vector<double>& rKeyStops ) :
-            maKeyStops(rKeyStops),
+        KeyStopLerp::KeyStopLerp( std::vector<double>&& rKeyStops ) :
+            maKeyStops(std::move(rKeyStops)),
             mnLastIndex(0)
         {
             validateInput(maKeyStops);
diff --git a/include/basegfx/utils/keystoplerp.hxx b/include/basegfx/utils/keystoplerp.hxx
index 302020ac337e..f8a821e7a58f 100644
--- a/include/basegfx/utils/keystoplerp.hxx
+++ b/include/basegfx/utils/keystoplerp.hxx
@@ -53,7 +53,7 @@ namespace basegfx::utils
                 need key stop lerping in the first place). All
                 elements must be of monotonically increasing value.
              */
-            explicit KeyStopLerp( const std::vector<double>& rKeyStops );
+            explicit KeyStopLerp( std::vector<double>&& rKeyStops );
 
             /** Create lerper with given sequence of stops
 
diff --git a/slideshow/source/engine/activities/continuouskeytimeactivitybase.cxx b/slideshow/source/engine/activities/continuouskeytimeactivitybase.cxx
index 38d3590da3b6..db25d2dd4333 100644
--- a/slideshow/source/engine/activities/continuouskeytimeactivitybase.cxx
+++ b/slideshow/source/engine/activities/continuouskeytimeactivitybase.cxx
@@ -28,7 +28,7 @@ namespace slideshow::internal
 {
         ContinuousKeyTimeActivityBase::ContinuousKeyTimeActivityBase( const ActivityParameters& rParms ) :
             SimpleContinuousActivityBase( rParms ),
-            maLerper( rParms.maDiscreteTimes )
+            maLerper( std::vector(rParms.maDiscreteTimes) )
         {
             ENSURE_OR_THROW( rParms.maDiscreteTimes.size() > 1,
                               "ContinuousKeyTimeActivityBase::ContinuousKeyTimeActivityBase(): key times vector must have two entries or more" );