D24395: Add some std::move in setter functions
Albert Astals Cid
noreply at phabricator.kde.org
Thu Oct 3 18:12:11 BST 2019
aacid created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
aacid requested review of this revision.
REVISION SUMMARY
typically we pass the param as const &, but here we forgot, we can fix
that problem by adding an std::move and have a very similar effect to if
we had declared the params as const &
REPOSITORY
R235 Attica
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D24395
AFFECTED FILES
src/comment.cpp
src/content.cpp
src/forum.cpp
src/knowledgebaseentry.cpp
To: aacid
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191003/321f1af0/attachment.html>
More information about the Kde-frameworks-devel
mailing list