Does not really matters in this case - loops are in const method already.
Othervice yeah - const auto & should be preferable. But we have relatively few places with non-const access (net pack appliers & initialization)