| commit | 65740868e3fc274865a21f421a6bed3bc192c2a4 | |
| author | Olly Betts <olly@survex.com> | |
| Sun, 23 Jul 2023 22:57:55 +0000 (24 10:57 +1200) | ||
| committer | Olly Betts <olly@survex.com> | |
| Mon, 24 Jul 2023 20:12:01 +0000 (25 08:12 +1200) | ||
| tree | a878ff8ce1836a1c8ac7b8e85205798a4738f60d | treesnapshot (tar.gz zip) |
| parent | 0577e36c8ef51776fbf5f453aeca5b55b1143aa1 | commitdiff |
Address GCC13 -Wredunant-move warnings
These std::move() were added in e23164b4ddb6cdc57136f4da2598d831f3780035
to fix clang warnings from -Wreturn-std-move (enabled by -Wall). Just
removing the std::move() reintroduces those warning with clang 8 and
clang 11 (but not clang 13 or later) but changing to apply a static_cast
to the returned type seems to make all versions of both compilers happy.
These std::move() were added in e23164b4ddb6cdc57136f4da2598d831f3780035
to fix clang warnings from -Wreturn-std-move (enabled by -Wall). Just
removing the std::move() reintroduces those warning with clang 8 and
clang 11 (but not clang 13 or later) but changing to apply a static_cast
to the returned type seems to make all versions of both compilers happy.
| xapian-core/tests/harness/backendmanager.cc | diffblobblamehistory | |
| xapian-core/tests/harness/backendmanager_inmemory.cc | diffblobblamehistory |