On my team, we take the extreme position that if
std::move
is used on something, treat it as if the rvalue reference was exploited. Even in situations that you know the rvalue was not exploited, such as in this case. (But in this case, it'd get dinged in the code review for being a purposeless no-op.) —
Eljay 50 secs ago