diff options
| author | Marc Mutz <[email protected]> | 2025-08-19 11:47:50 +0200 |
|---|---|---|
| committer | Marc Mutz <[email protected]> | 2025-08-20 11:44:29 +0200 |
| commit | a31a8da004c3381c5a0aa2acf5c0a2d5eb33f427 (patch) | |
| tree | 9c7025d5c9ea5527b19f24b139ac1df30eb1efc6 /src/dbus/qdbusconnectionmanager.cpp | |
| parent | eaeeb6b5d38afc3ef249dfd827dad1181bef14d2 (diff) | |
On GCC, this function is NRVO'd, but depending on the definition of
va_list, a compiler may be unable to NRVO this function, in which case
the const on 's' prevents the implicit return by move.
Found by Coverity.
Fix by dropping the const. It doesn't hurt readability in such a short
function, shuts up Coverity, and may improve codegen on some
platforms.
Amends d251cae7b7370af328307948aca9bb63def22fe4.
Coverity-Id: 474158
Pick-to: 6.10 6.9 6.8
Change-Id: I4f842f690ea45c8db4c02f32f858d2700f57d128
Reviewed-by: Edward Welbourne <[email protected]>
Diffstat (limited to 'src/dbus/qdbusconnectionmanager.cpp')
0 files changed, 0 insertions, 0 deletions