diff options
author | Marc Mutz <[email protected]> | 2025-07-01 11:15:55 +0200 |
---|---|---|
committer | Marc Mutz <[email protected]> | 2025-07-02 11:32:28 +0000 |
commit | 04b235d0120105b901a185e6dbc74c2c5941ce18 (patch) | |
tree | ed5019bfdfd5f5db0079c8a61b0ff886fd81c34d | |
parent | 76d374d138a0824bc6f438d47c3bbd6ce998c2ed (diff) |
Coverity complains about a missing std::move(), because the
QDockAreaLayout streaming operator (like all non-QDebug-member ones)
takes the QDebug stream by value.
Add the missing move().
The const char* stream operator is a QDebug member, so takes the
stream by reference, so no std::move() needed.
It's a bit sad that it has come to this, but c'est la vie...
Amends 1cd7e01c909499b4c4c6d7dd250cc67bbb962141.
Coverity-Id: 481489
Pick-to: 6.10
Change-Id: Ifed3a92e3db9eb21edf14bf79ba0c860311f006a
Reviewed-by: Axel Spoerl <[email protected]>
-rw-r--r-- | src/widgets/widgets/qmainwindowlayout.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/widgets/widgets/qmainwindowlayout.cpp b/src/widgets/widgets/qmainwindowlayout.cpp index ed6f6f6193b..87c33932a8d 100644 --- a/src/widgets/widgets/qmainwindowlayout.cpp +++ b/src/widgets/widgets/qmainwindowlayout.cpp @@ -147,7 +147,7 @@ QDebug operator<<(QDebug debug, const QDockAreaLayout &layout) QDebug operator<<(QDebug debug, const QMainWindowLayout *layout) { if (layout) - return debug << layout->layoutState.dockAreaLayout; + return std::move(debug) << layout->layoutState.dockAreaLayout; return debug << "QMainWindowLayout(0x0)"; } |