as you're taking a copy, there's no reason for this method to be const char *const other than redirection.  Just leave it as std::string and be done with it.