Page MenuHomePhabricator

[services] Get rid of `return std::move(...)` in services codebase
ClosedPublic

Authored by atul on Nov 10 2022, 2:03 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jun 21, 12:41 AM
Unknown Object (File)
Fri, Jun 14, 7:05 AM
Unknown Object (File)
Tue, Jun 11, 12:23 PM
Unknown Object (File)
Jun 3 2024, 3:17 PM
Unknown Object (File)
May 18 2024, 11:36 AM
Unknown Object (File)
Apr 23 2024, 4:01 AM
Unknown Object (File)
Apr 23 2024, 4:01 AM
Unknown Object (File)
Apr 23 2024, 4:01 AM
Subscribers

Details

Summary

Context: https://linear.app/comm/issue/ENG-619/get-rid-of-stdmove-in-the-services-codebase

https://developers.redhat.com/blog/2019/04/12/understanding-when-not-to-stdmove-in-c
https://phabricator.ashoat.com/D2912?id=8614#inline-15820
https://phabricator.ashoat.com/D2876?id=8631#inline-15961

NOTE: We don't want to remove std::move altogether, just when we're returning so we don't interfere with return value optimization. We're still going to need to std::move unique_ptrs into lambdas and whatnot.
Test Plan

CI and close reading and previous discussion

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable