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)
Mon, Sep 23, 4:53 AM
Unknown Object (File)
Thu, Sep 19, 6:20 AM
Unknown Object (File)
Thu, Sep 19, 6:20 AM
Unknown Object (File)
Thu, Sep 19, 6:20 AM
Unknown Object (File)
Thu, Sep 19, 6:16 AM
Unknown Object (File)
Aug 20 2024, 7:15 PM
Unknown Object (File)
Jul 22 2024, 8:45 PM
Unknown Object (File)
Jul 20 2024, 1:03 PM
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