Pass std::functions as const& to avoid copy for each invocation

We have plenty of places where we pass a std::function by value for later just calling it. Pass by const reference to avoid the copy