remove unnecessary std::move; pass vector or string arguments by reference
Prefer C(const X& x) { m_x = x; }
over C(X x) { m_x = std::move(x); }
.
In the past, we changes style forth and back, but the Cpp Core Guidelines F15 etc is pretty clear that in most circumstances "normal parameter passing" is the best.
Undecided yet whether this also applies to OwningVector
.
Edited by Wuttke, Joachim