diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2022-08-23 15:37:06 +0200 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2022-08-23 17:24:34 +0200 |
| commit | 4c27aaa787a67e05a592adcf6cf42e49b7a02eaf (patch) | |
| tree | 8f46879bb406acd5637390016b94f659e16bbd34 /slideshow | |
| parent | 783f166793915a1c5a008de7142f773ad3898683 (diff) | |
-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 <sbergman@redhat.com>
Diffstat (limited to 'slideshow')
| -rw-r--r-- | slideshow/source/engine/eventqueue.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
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', |