diff options
author | 2017-08-08 08:38:36 -0700 | |
---|---|---|
committer | 2017-08-08 08:39:28 -0700 | |
commit | f8e58ee1d02d2476ae92ebc3737e42b8b6a36891 (patch) | |
tree | 8ce8590b9c5cc08fc3183b8c41f3008ca4088a03 | |
parent | 1288dd799f3502f7201fc88d93ae69a1869d06fa (diff) | |
download | paludis-f8e58ee1d02d2476ae92ebc3737e42b8b6a36891.tar.gz paludis-f8e58ee1d02d2476ae92ebc3737e42b8b6a36891.tar.xz |
resolver: use range based for loops (NFC)
Convert a few loops to range based for loops instead of standard for
loops.
-rw-r--r-- | paludis/resolver/get_sameness.cc | 24 |
1 files changed, 10 insertions, 14 deletions
diff --git a/paludis/resolver/get_sameness.cc b/paludis/resolver/get_sameness.cc index 4a07ba6c7..22d0abdf9 100644 --- a/paludis/resolver/get_sameness.cc +++ b/paludis/resolver/get_sameness.cc @@ -155,28 +155,24 @@ paludis::resolver::get_sameness( existing_choices = existing_id->choices_key()->parse_value(); std::set<ChoiceNameWithPrefix> i_common, u_common; - for (Choices::ConstIterator k(installable_choices->begin()), k_end(installable_choices->end()) ; - k != k_end ; ++k) + for (const auto & c : *installable_choices) { - if (! (*k)->consider_added_or_changed()) + if (! c->consider_added_or_changed()) continue; - for (Choice::ConstIterator i((*k)->begin()), i_end((*k)->end()) ; - i != i_end ; ++i) - if (co_explicit == (*i)->origin()) - i_common.insert((*i)->name_with_prefix()); + for (const auto & v : *c) + if (co_explicit == v->origin()) + i_common.insert(v->name_with_prefix()); } - for (Choices::ConstIterator k(existing_choices->begin()), k_end(existing_choices->end()) ; - k != k_end ; ++k) + for (const auto & c : *existing_choices) { - if (! (*k)->consider_added_or_changed()) + if (! c->consider_added_or_changed()) continue; - for (Choice::ConstIterator i((*k)->begin()), i_end((*k)->end()) ; - i != i_end ; ++i) - if (co_explicit == (*i)->origin()) - u_common.insert((*i)->name_with_prefix()); + for (const auto & v : *c) + if (co_explicit == v->origin()) + u_common.insert(v->name_with_prefix()); } std::set_intersection( |