diff --git a/src/units/filter.cpp b/src/units/filter.cpp index dcc33f79afe..328664456a4 100644 --- a/src/units/filter.cpp +++ b/src/units/filter.cpp @@ -71,7 +71,7 @@ std::vector unit_filter::all_matches_on_map(const map_location* lo unit_const_ptr unit_filter::first_match_on_map() const { const unit_map & units = fc_->get_disp_context().units(); - for(unit_map::const_iterator u = units.begin(); u != units.end(); u++) { + for(unit_map::const_iterator u = units.begin(); u != units.end(); ++u) { if (matches(*u, u->get_location())) { return u.get_shared_ptr(); } diff --git a/src/utils/name_generator_factory.cpp b/src/utils/name_generator_factory.cpp index c16adb7fd6d..5966139ab5a 100644 --- a/src/utils/name_generator_factory.cpp +++ b/src/utils/name_generator_factory.cpp @@ -26,7 +26,7 @@ std::string name_generator::generate(const std::map& va name_generator_factory::name_generator_factory(const config& config, std::vector ids) : name_generators_() { add_name_generator_from_config(config, "", ""); - for (std::vector::iterator it = std::begin(ids); it!=std::end(ids); it++) { + for (std::vector::iterator it = std::begin(ids); it!=std::end(ids); ++it) { std::string id = *it; add_name_generator_from_config(config, id, (id + "_")); } diff --git a/src/whiteboard/highlighter.cpp b/src/whiteboard/highlighter.cpp index 7b954b1b751..026bb6e321a 100644 --- a/src/whiteboard/highlighter.cpp +++ b/src/whiteboard/highlighter.cpp @@ -196,7 +196,7 @@ void highlighter::last_action_redraw(move_ptr move) { // Note: the planned actions seem to only get removed from the screen when // a redraw is triggered by the mouse cursor moving over them. - for (side_actions::iterator iterator = sa.begin(); iterator < sa.end(); iterator++) + for (side_actions::iterator iterator = sa.begin(); iterator < sa.end(); ++iterator) { if (iterator->get()->get_unit() == nullptr) sa.remove_action (iterator); diff --git a/src/whiteboard/utility.cpp b/src/whiteboard/utility.cpp index 36408ed8400..55eb3424efe 100644 --- a/src/whiteboard/utility.cpp +++ b/src/whiteboard/utility.cpp @@ -65,7 +65,7 @@ unit_const_ptr find_backup_leader(const unit & leader) { assert(leader.can_recruit()); assert(resources::gameboard->map().is_keep(leader.get_location())); - for (unit_map::const_iterator unit = resources::gameboard->units().begin(); unit != resources::gameboard->units().end(); unit++) + for (unit_map::const_iterator unit = resources::gameboard->units().begin(); unit != resources::gameboard->units().end(); ++unit) { if (unit->can_recruit() && unit->id() != leader.id()) {