diff options
author | Marc Mutz <marc.mutz@qt.io> | 2022-01-22 21:18:15 +0100 |
---|---|---|
committer | Marc Mutz <marc.mutz@qt.io> | 2022-01-24 21:11:22 +0100 |
commit | a8f00bf0ad864afc25986c0132d3397d785eb9fe (patch) | |
tree | 956bf2bf0c7cde022ab84382b1026b6aae2803a9 /src/corelib/configure.cmake | |
parent | 6f1821dfa302c028f7899e4485dfaf24a6a02457 (diff) |
QByteArray benchmark: fix clang -Wmove-result
Take the argument by value instead of rvalue ref. In C++17, this is as
efficient, and enables the
return big;
to use the QByteArray move constructor, avoiding the need for a manual
std::move(), which always looks suspicious on a return statement,
because more often than not, it's a pessimization that breaks NRVO.
This code doesn't seem to exist in Qt 6.2, so only
Pick-to: 6.3
Change-Id: I8bf678102f5df1870cfc61090d12f327478d74d1
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
Diffstat (limited to 'src/corelib/configure.cmake')
0 files changed, 0 insertions, 0 deletions