slideshow/source/engine/eventqueue.cxx | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
New commits:
commit 4c27aaa787a67e05a592adcf6cf42e49b7a02eaf
Author: Stephan Bergmann <sberg...@redhat.com>
AuthorDate: Tue Aug 23 15:37:06 2022 +0200
Commit: Stephan Bergmann <sberg...@redhat.com>
CommitDate: Tue Aug 23 17:24:34 2022 +0200
-Werror=redundant-move
...with recent GCC 16 trunk. Introduced with
1a2926a995fdbdcdae0ca6407877084f3520e539 "use std::move when popping stuff
off
stacks" but which doesn't make sense here where std::priority_queue's top()
unconditionally returns a const reference (unlike std::stack's top()).
Change-Id: I408920220e3b7ffe775fa87f9edbfdc7721058db
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138732
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sberg...@redhat.com>
diff --git a/slideshow/source/engine/eventqueue.cxx
b/slideshow/source/engine/eventqueue.cxx
index 4a46a0c806b6..86d58a5fe6b2 100644
--- a/slideshow/source/engine/eventqueue.cxx
+++ b/slideshow/source/engine/eventqueue.cxx
@@ -172,7 +172,7 @@ namespace slideshow::internal
&& !bFireAllEvents
&& (maEvents.empty() || maEvents.top().nTime > nCurrTime))
{
- const EventEntry aEvent (std::move(maNextNextEvents.top()));
+ const EventEntry aEvent (maNextNextEvents.top());
maNextNextEvents.pop();
maEvents.push(aEvent);
}
@@ -184,7 +184,7 @@ namespace slideshow::internal
while( !maEvents.empty() &&
(bFireAllEvents || maEvents.top().nTime <= nCurrTime) )
{
- EventEntry event( std::move(maEvents.top()) );
+ EventEntry event( maEvents.top() );
maEvents.pop();
// only process event, if it is still 'charged',
