get rid of const & std::string pattern

[jeanlauliac]

Instead take strings just as std::string, and std::move then locally. That way if the call site uses an rvalue, no copy is ever done, it's moved all the way.