Is the compiler smart enough not to call the `std::string` constructor every time we call this function?  Should we have `const` strings ready for it to return?