Make base set_name take const std:string& instead of std:string Summary: All the subclasses do that so it causes warnings not to and is dumb because copy constructing std::string is dumb Reviewed By: kholst Test Plan: Generate some codes, compile with -Werror