we tend to prefer std::string, is there a reason why not to here?