Avoid Objects with Names

// Bad
MyType x{42, "hello"};
foo(x);

// Good
foo({42,"hellow"});

When You Cannot Avoid Using Names

std::string str{"hello"};
std::vector<std::string> coll;

coll.push_back(str);
coll.push_back(std::move(str));

std::string line;
while(std::getline(myStream, line)) {
   coll.push_back(std::move(line));
}

Avoid Unnecessary std::move()

• If the returned expression is not an lvalue, the NRVO may be inhibited:

   std::string foo(){
      std::string name;
      ...
      // BAD:
      return std::move(name);
   }
  

  See Name Returned Value Optimization for more details.

• Thus, if you return a local object, do not use std::move() whereby either NRVO or move semantics will perform:

  std::string foo(){
     std::string name;
     ...
     return name;
  }