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