diff options
author | Caolán McNamara <caolan.mcnamara@collabora.com> | 2023-12-23 14:13:05 +0000 |
---|---|---|
committer | Caolán McNamara <caolan.mcnamara@collabora.com> | 2023-12-23 19:52:50 +0100 |
commit | 9a6298aedb663513c039efccc516dec3188dd8fa (patch) | |
tree | 068a77e35769a4ba0de3aa66b09d1a467ba31faa /slideshow | |
parent | f82b8bf05a76ad9cc3fe950f4180c90bce56d79b (diff) |
cid#1546006 COPY_INSTEAD_OF_MOVE
and
cid#1545999 COPY_INSTEAD_OF_MOVE
cid#1545995 COPY_INSTEAD_OF_MOVE
cid#1545994 COPY_INSTEAD_OF_MOVE
cid#1545989 COPY_INSTEAD_OF_MOVE
cid#1545982 COPY_INSTEAD_OF_MOVE
cid#1545977 COPY_INSTEAD_OF_MOVE
cid#1545970 COPY_INSTEAD_OF_MOVE
cid#1545856 COPY_INSTEAD_OF_MOVE
cid#1545845 COPY_INSTEAD_OF_MOVE
cid#1545838 COPY_INSTEAD_OF_MOVE
cid#1545776 COPY_INSTEAD_OF_MOVE
cid#1545774 COPY_INSTEAD_OF_MOVE
cid#1545744 COPY_INSTEAD_OF_MOVE
cid#1545719 COPY_INSTEAD_OF_MOVE
cid#1545716 COPY_INSTEAD_OF_MOVE
cid#1545687 COPY_INSTEAD_OF_MOVE
cid#1545648 COPY_INSTEAD_OF_MOVE
cid#1545643 COPY_INSTEAD_OF_MOVE
cid#1545641 COPY_INSTEAD_OF_MOVE
cid#1545604 COPY_INSTEAD_OF_MOVE
cid#1545531 COPY_INSTEAD_OF_MOVE
cid#1545530 COPY_INSTEAD_OF_MOVE
cid#1545524 COPY_INSTEAD_OF_MOVE
cid#1545516 COPY_INSTEAD_OF_MOVE
cid#1545501 COPY_INSTEAD_OF_MOVE
cid#1545486 COPY_INSTEAD_OF_MOVE
Change-Id: Idcf3e27bb9e81b6a77d5ef75c9376eb1d6aff8c9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/161245
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Diffstat (limited to 'slideshow')
3 files changed, 6 insertions, 6 deletions
diff --git a/slideshow/source/engine/animationnodes/animationaudionode.cxx b/slideshow/source/engine/animationnodes/animationaudionode.cxx index 1dcd4cf7181b..a548fd20f97e 100644 --- a/slideshow/source/engine/animationnodes/animationaudionode.cxx +++ b/slideshow/source/engine/animationnodes/animationaudionode.cxx @@ -89,7 +89,7 @@ void AnimationAudioNode::activate_st() // deactivate ASAP: auto self(getSelf()); scheduleDeactivationEvent( - makeEvent( [self] () { self->deactivate(); }, + makeEvent( [self=std::move(self)] () { self->deactivate(); }, "AnimationAudioNode::deactivate without delay") ); } } @@ -188,13 +188,13 @@ bool AnimationAudioNode::handleAnimationEvent( void AnimationAudioNode::checkPlayingStatus() { - auto self(getSelf()); + auto xSelf(getSelf()); double nDuration = mpPlayer->getDuration(); if (!mpPlayer->isPlaying() || nDuration < 0.0) nDuration = 0.0; scheduleDeactivationEvent( - makeDelay( [self] () { self->deactivate(); }, + makeDelay( [xSelf=std::move(xSelf)] () { xSelf->deactivate(); }, nDuration, "AnimationAudioNode::deactivate with delay") ); } diff --git a/slideshow/source/engine/animationnodes/basenode.cxx b/slideshow/source/engine/animationnodes/basenode.cxx index 9e812ecfc794..1da302008138 100644 --- a/slideshow/source/engine/animationnodes/basenode.cxx +++ b/slideshow/source/engine/animationnodes/basenode.cxx @@ -462,7 +462,7 @@ bool BaseNode::resolve() if (aBegin.hasValue()) { auto self(mpSelf); mpCurrentEvent = generateEvent( - aBegin, [self] () { self->activate(); }, + aBegin, [self=std::move(self)] () { self->activate(); }, maContext, mnStartDelay ); } else { @@ -474,7 +474,7 @@ bool BaseNode::resolve() // timeout into account auto self(mpSelf); mpCurrentEvent = makeDelay( - [self] () { self->activate(); }, + [self=std::move(self)] () { self->activate(); }, mnStartDelay, "AnimationNode::activate with delay"); maContext.mrEventQueue.addEvent( mpCurrentEvent ); diff --git a/slideshow/source/engine/animationnodes/sequentialtimecontainer.cxx b/slideshow/source/engine/animationnodes/sequentialtimecontainer.cxx index 3f928593c846..5b1eecf79d31 100644 --- a/slideshow/source/engine/animationnodes/sequentialtimecontainer.cxx +++ b/slideshow/source/engine/animationnodes/sequentialtimecontainer.cxx @@ -43,7 +43,7 @@ void SequentialTimeContainer::activate_st() // deactivate ASAP: auto self(getSelf()); scheduleDeactivationEvent( - makeEvent( [self] () { self->deactivate(); }, + makeEvent( [self=std::move(self)] () { self->deactivate(); }, "SequentialTimeContainer::deactivate") ); } else // use default |