rethink passing `std::string` by value

[attilammagyar]

I had the impression that move constructors, copy elision, small-string optimization, and other modern C++ features will take care of it, but as @ZonderP pointed out in issue #6, I never actually confirmed these assumptions. Maybe passing around string_view-s or const refs would actually be beneficial for most parts of the code.