Made some method params const reference, mostly std::strings

Found and auto-fixed by clang-tidy’s performance-unnecessary-value-param check.

The check had other suggestions too, but I limited it to these ones, which are easier to reason about.