It's better to use `std::string()` rather than `""` if you want an empty std::string.  E.g. with GCC, there's a shared empty string object internals which std::string() gives you, whereas using `""` forces creating a new one.

Admittedly it's not hugely important here as the code is never actually executed...