Review usage of std::string and switch to std::string_view whenever possible

[cschreib]

I haven't spotted any obvious spot that isn't already using this (string conversion)