]> git.lizzy.rs Git - minetest.git/blobdiff - src/serverenvironment.cpp
Code modernization: subfolders (#6283)
[minetest.git] / src / serverenvironment.cpp
index 161b24fd4f86698e3e9ca0c8ae5f5c40cbc66227..04015d953bd3418ce6fdef2b4f1cb7fa0750d34b 100644 (file)
@@ -21,6 +21,7 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #include "content_sao.h"
 #include "settings.h"
 #include "log.h"
+#include "mapblock.h"
 #include "nodedef.h"
 #include "nodemetadata.h"
 #include "gamedef.h"
@@ -30,7 +31,6 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #include "remoteplayer.h"
 #include "scripting_server.h"
 #include "server.h"
-#include "voxelalgorithms.h"
 #include "util/serialize.h"
 #include "util/basic_macros.h"
 #include "util/pointedthing.h"
@@ -54,8 +54,7 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 */
 
 ABMWithState::ABMWithState(ActiveBlockModifier *abm_):
-       abm(abm_),
-       timer(0)
+       abm(abm_)
 {
        // Initialize timer to random value to spread processing
        float itv = abm->getTriggerInterval();
@@ -71,9 +70,8 @@ ABMWithState::ABMWithState(ActiveBlockModifier *abm_):
 
 void LBMContentMapping::deleteContents()
 {
-       for (std::vector<LoadingBlockModifierDef *>::iterator it = lbm_list.begin();
-               it != lbm_list.end(); ++it) {
-               delete *it;
+       for (auto &it : lbm_list) {
+               delete it;
        }
 }
 
@@ -85,24 +83,21 @@ void LBMContentMapping::addLBM(LoadingBlockModifierDef *lbm_def, IGameDef *gamed
 
        lbm_list.push_back(lbm_def);
 
-       for (std::set<std::string>::const_iterator it = lbm_def->trigger_contents.begin();
-               it != lbm_def->trigger_contents.end(); ++it) {
+       for (const std::string &nodeTrigger: lbm_def->trigger_contents) {
                std::set<content_t> c_ids;
-               bool found = nodedef->getIds(*it, c_ids);
+               bool found = nodedef->getIds(nodeTrigger, c_ids);
                if (!found) {
-                       content_t c_id = gamedef->allocateUnknownNodeId(*it);
+                       content_t c_id = gamedef->allocateUnknownNodeId(nodeTrigger);
                        if (c_id == CONTENT_IGNORE) {
                                // Seems it can't be allocated.
-                               warningstream << "Could not internalize node name \"" << *it
+                               warningstream << "Could not internalize node name \"" << nodeTrigger
                                        << "\" while loading LBM \"" << lbm_def->name << "\"." << std::endl;
                                continue;
                        }
                        c_ids.insert(c_id);
                }
 
-               for (std::set<content_t>::const_iterator iit =
-                       c_ids.begin(); iit != c_ids.end(); ++iit) {
-                       content_t c_id = *iit;
+               for (content_t c_id : c_ids) {
                        map[c_id].push_back(lbm_def);
                }
        }
@@ -111,7 +106,7 @@ void LBMContentMapping::addLBM(LoadingBlockModifierDef *lbm_def, IGameDef *gamed
 const std::vector<LoadingBlockModifierDef *> *
 LBMContentMapping::lookup(content_t c) const
 {
-       container_map::const_iterator it = map.find(c);
+       lbm_map::const_iterator it = map.find(c);
        if (it == map.end())
                return NULL;
        // This first dereferences the iterator, returning
@@ -122,20 +117,19 @@ LBMContentMapping::lookup(content_t c) const
 
 LBMManager::~LBMManager()
 {
-       for (std::map<std::string, LoadingBlockModifierDef *>::iterator it =
-               m_lbm_defs.begin(); it != m_lbm_defs.end(); ++it) {
-               delete it->second;
+       for (auto &m_lbm_def : m_lbm_defs) {
+               delete m_lbm_def.second;
        }
-       for (lbm_lookup_map::iterator it = m_lbm_lookup.begin();
-               it != m_lbm_lookup.end(); ++it) {
-               (it->second).deleteContents();
+
+       for (auto &it : m_lbm_lookup) {
+               (it.second).deleteContents();
        }
 }
 
 void LBMManager::addLBMDef(LoadingBlockModifierDef *lbm_def)
 {
        // Precondition, in query mode the map isn't used anymore
-       FATAL_ERROR_IF(m_query_mode == true,
+       FATAL_ERROR_IF(m_query_mode,
                "attempted to modify LBMManager in query mode");
 
        if (!string_allowed(lbm_def->name, LBM_NAME_ALLOWED_CHARS)) {
@@ -166,7 +160,7 @@ void LBMManager::loadIntroductionTimes(const std::string &times,
 
        size_t idx = 0;
        size_t idx_new;
-       while ((idx_new = times.find(";", idx)) != std::string::npos) {
+       while ((idx_new = times.find(';', idx)) != std::string::npos) {
                std::string entry = times.substr(idx, idx_new - idx);
                std::vector<std::string> components = str_split(entry, '~');
                if (components.size() != 2)
@@ -214,12 +208,11 @@ void LBMManager::loadIntroductionTimes(const std::string &times,
        LBMContentMapping &lbms_we_introduce_now = m_lbm_lookup[now];
        LBMContentMapping &lbms_running_always = m_lbm_lookup[U32_MAX];
 
-       for (std::map<std::string, LoadingBlockModifierDef *>::iterator it =
-               m_lbm_defs.begin(); it != m_lbm_defs.end(); ++it) {
-               if (it->second->run_at_every_load) {
-                       lbms_running_always.addLBM(it->second, gamedef);
+       for (auto &m_lbm_def : m_lbm_defs) {
+               if (m_lbm_def.second->run_at_every_load) {
+                       lbms_running_always.addLBM(m_lbm_def.second, gamedef);
                } else {
-                       lbms_we_introduce_now.addLBM(it->second, gamedef);
+                       lbms_we_introduce_now.addLBM(m_lbm_def.second, gamedef);
                }
        }
 
@@ -231,22 +224,20 @@ void LBMManager::loadIntroductionTimes(const std::string &times,
 std::string LBMManager::createIntroductionTimesString()
 {
        // Precondition, we must be in query mode
-       FATAL_ERROR_IF(m_query_mode == false,
+       FATAL_ERROR_IF(!m_query_mode,
                "attempted to query on non fully set up LBMManager");
 
        std::ostringstream oss;
-       for (lbm_lookup_map::iterator it = m_lbm_lookup.begin();
-               it != m_lbm_lookup.end(); ++it) {
-               u32 time = it->first;
-               std::vector<LoadingBlockModifierDef *> &lbm_list = it->second.lbm_list;
-               for (std::vector<LoadingBlockModifierDef *>::iterator iit = lbm_list.begin();
-                       iit != lbm_list.end(); ++iit) {
+       for (const auto &it : m_lbm_lookup) {
+               u32 time = it.first;
+               const std::vector<LoadingBlockModifierDef *> &lbm_list = it.second.lbm_list;
+               for (const auto &lbm_def : lbm_list) {
                        // Don't add if the LBM runs at every load,
                        // then introducement time is hardcoded
                        // and doesn't need to be stored
-                       if ((*iit)->run_at_every_load)
+                       if (lbm_def->run_at_every_load)
                                continue;
-                       oss << (*iit)->name << "~" << time << ";";
+                       oss << lbm_def->name << "~" << time << ";";
                }
        }
        return oss.str();
@@ -255,7 +246,7 @@ std::string LBMManager::createIntroductionTimesString()
 void LBMManager::applyLBMs(ServerEnvironment *env, MapBlock *block, u32 stamp)
 {
        // Precondition, we need m_lbm_lookup to be initialized
-       FATAL_ERROR_IF(m_query_mode == false,
+       FATAL_ERROR_IF(!m_query_mode,
                "attempted to query on non fully set up LBMManager");
        v3s16 pos_of_block = block->getPosRelative();
        v3s16 pos;
@@ -274,9 +265,8 @@ void LBMManager::applyLBMs(ServerEnvironment *env, MapBlock *block, u32 stamp)
                                                iit->second.lookup(c);
                                        if (!lbm_list)
                                                continue;
-                                       for (std::vector<LoadingBlockModifierDef *>::const_iterator iit =
-                                               lbm_list->begin(); iit != lbm_list->end(); ++iit) {
-                                               (*iit)->trigger(env, pos + pos_of_block, n);
+                                       for (auto lbmdef : *lbm_list) {
+                                               lbmdef->trigger(env, pos + pos_of_block, n);
                                        }
                                }
                        }
@@ -310,22 +300,17 @@ void ActiveBlockList::update(std::vector<v3s16> &active_positions,
                Create the new list
        */
        std::set<v3s16> newlist = m_forceloaded_list;
-       for(std::vector<v3s16>::iterator i = active_positions.begin();
-               i != active_positions.end(); ++i)
-       {
-               fillRadiusBlock(*i, radius, newlist);
+       for (const v3s16 &active_position : active_positions) {
+               fillRadiusBlock(active_position, radius, newlist);
        }
 
        /*
                Find out which blocks on the old list are not on the new list
        */
        // Go through old list
-       for(std::set<v3s16>::iterator i = m_list.begin();
-               i != m_list.end(); ++i)
-       {
-               v3s16 p = *i;
+       for (v3s16 p : m_list) {
                // If not on new list, it's been removed
-               if(newlist.find(p) == newlist.end())
+               if (newlist.find(p) == newlist.end())
                        blocks_removed.insert(p);
        }
 
@@ -333,10 +318,7 @@ void ActiveBlockList::update(std::vector<v3s16> &active_positions,
                Find out which blocks on the new list are not on the old list
        */
        // Go through new list
-       for(std::set<v3s16>::iterator i = newlist.begin();
-               i != newlist.end(); ++i)
-       {
-               v3s16 p = *i;
+       for (v3s16 p : newlist) {
                // If not on old list, it's been added
                if(m_list.find(p) == m_list.end())
                        blocks_added.insert(p);
@@ -346,10 +328,7 @@ void ActiveBlockList::update(std::vector<v3s16> &active_positions,
                Update m_list
        */
        m_list.clear();
-       for(std::set<v3s16>::iterator i = newlist.begin();
-               i != newlist.end(); ++i)
-       {
-               v3s16 p = *i;
+       for (v3s16 p : newlist) {
                m_list.insert(p);
        }
 }
@@ -365,15 +344,7 @@ ServerEnvironment::ServerEnvironment(ServerMap *map,
        m_map(map),
        m_script(scriptIface),
        m_server(server),
-       m_path_world(path_world),
-       m_send_recommended_timer(0),
-       m_active_block_interval_overload_skip(0),
-       m_game_time(0),
-       m_game_time_fraction_counter(0),
-       m_last_clear_objects_time(0),
-       m_recommended_send_interval(0.1),
-       m_max_lag_estimate(0.1),
-       m_player_database(NULL)
+       m_path_world(path_world)
 {
        // Determine which database backend to use
        std::string conf_path = path_world + DIR_DELIM + "world.mt";
@@ -393,7 +364,7 @@ ServerEnvironment::ServerEnvironment(ServerMap *map,
                }
        }
 
-       std::string name = "";
+       std::string name;
        conf.getNoEx("player_backend", name);
        m_player_database = openPlayerDatabase(name, path_world, conf);
 }
@@ -411,15 +382,13 @@ ServerEnvironment::~ServerEnvironment()
        m_map->drop();
 
        // Delete ActiveBlockModifiers
-       for (std::vector<ABMWithState>::iterator
-               i = m_abms.begin(); i != m_abms.end(); ++i){
-               delete i->abm;
+       for (ABMWithState &m_abm : m_abms) {
+               delete m_abm.abm;
        }
 
        // Deallocate players
-       for (std::vector<RemotePlayer *>::iterator i = m_players.begin();
-               i != m_players.end(); ++i) {
-               delete (*i);
+       for (RemotePlayer *m_player : m_players) {
+               delete m_player;
        }
 
        delete m_player_database;
@@ -437,9 +406,7 @@ ServerMap & ServerEnvironment::getServerMap()
 
 RemotePlayer *ServerEnvironment::getPlayer(const u16 peer_id)
 {
-       for (std::vector<RemotePlayer *>::iterator i = m_players.begin();
-               i != m_players.end(); ++i) {
-               RemotePlayer *player = *i;
+       for (RemotePlayer *player : m_players) {
                if (player->peer_id == peer_id)
                        return player;
        }
@@ -448,9 +415,7 @@ RemotePlayer *ServerEnvironment::getPlayer(const u16 peer_id)
 
 RemotePlayer *ServerEnvironment::getPlayer(const char* name)
 {
-       for (std::vector<RemotePlayer *>::iterator i = m_players.begin();
-               i != m_players.end(); ++i) {
-               RemotePlayer *player = *i;
+       for (RemotePlayer *player : m_players) {
                if (strcmp(player->getName(), name) == 0)
                        return player;
        }
@@ -521,9 +486,7 @@ bool ServerEnvironment::line_of_sight(v3f pos1, v3f pos2, float stepsize, v3s16
 void ServerEnvironment::kickAllPlayers(AccessDeniedCode reason,
        const std::string &str_reason, bool reconnect)
 {
-       for (std::vector<RemotePlayer *>::iterator it = m_players.begin();
-               it != m_players.end(); ++it) {
-               RemotePlayer *player = dynamic_cast<RemotePlayer *>(*it);
+       for (RemotePlayer *player : m_players) {
                m_server->DenyAccessVerCompliant(player->peer_id,
                        player->protocol_version, reason, str_reason, reconnect);
        }
@@ -534,15 +497,13 @@ void ServerEnvironment::saveLoadedPlayers()
        std::string players_path = m_path_world + DIR_DELIM + "players";
        fs::CreateDir(players_path);
 
-       for (std::vector<RemotePlayer *>::iterator it = m_players.begin();
-               it != m_players.end();
-               ++it) {
-               if ((*it)->checkModified() ||
-                       ((*it)->getPlayerSAO() && (*it)->getPlayerSAO()->extendedAttributesModified())) {
+       for (RemotePlayer *player : m_players) {
+               if (player->checkModified() || (player->getPlayerSAO() &&
+                               player->getPlayerSAO()->extendedAttributesModified())) {
                        try {
-                               m_player_database->savePlayer(*it);
+                               m_player_database->savePlayer(player);
                        } catch (DatabaseException &e) {
-                               errorstream << "Failed to save player " << (*it)->getName() << " exception: "
+                               errorstream << "Failed to save player " << player->getName() << " exception: "
                                        << e.what() << std::endl;
                                throw;
                        }
@@ -652,14 +613,14 @@ void ServerEnvironment::loadMeta()
        }
 
        setTimeOfDay(args.exists("time_of_day") ?
-               // set day to morning by default
-               args.getU64("time_of_day") : 9000);
+               // set day to early morning by default
+               args.getU64("time_of_day") : 5250);
 
        m_last_clear_objects_time = args.exists("last_clear_objects_time") ?
                // If missing, do as if clearObjects was never called
                args.getU64("last_clear_objects_time") : 0;
 
-       std::string lbm_introduction_times = "";
+       std::string lbm_introduction_times;
        try {
                u64 ver = args.getU64("lbm_introduction_times_version");
                if (ver == 1) {
@@ -703,18 +664,17 @@ class ABMHandler
                if(dtime_s < 0.001)
                        return;
                INodeDefManager *ndef = env->getGameDef()->ndef();
-               for(std::vector<ABMWithState>::iterator
-                       i = abms.begin(); i != abms.end(); ++i) {
-                       ActiveBlockModifier *abm = i->abm;
+               for (ABMWithState &abmws : abms) {
+                       ActiveBlockModifier *abm = abmws.abm;
                        float trigger_interval = abm->getTriggerInterval();
                        if(trigger_interval < 0.001)
                                trigger_interval = 0.001;
                        float actual_interval = dtime_s;
                        if(use_timers){
-                               i->timer += dtime_s;
-                               if(i->timer < trigger_interval)
+                               abmws.timer += dtime_s;
+                               if(abmws.timer < trigger_interval)
                                        continue;
-                               i->timer -= trigger_interval;
+                               abmws.timer -= trigger_interval;
                                actual_interval = trigger_interval;
                        }
                        float chance = abm->getTriggerChance();
@@ -736,20 +696,16 @@ class ABMHandler
                        // Trigger neighbors
                        const std::set<std::string> &required_neighbors_s =
                                abm->getRequiredNeighbors();
-                       for (std::set<std::string>::iterator rn = required_neighbors_s.begin();
-                                       rn != required_neighbors_s.end(); ++rn) {
-                               ndef->getIds(*rn, aabm.required_neighbors);
+                       for (const std::string &required_neighbor_s : required_neighbors_s) {
+                               ndef->getIds(required_neighbor_s, aabm.required_neighbors);
                        }
 
                        // Trigger contents
                        const std::set<std::string> &contents_s = abm->getTriggerContents();
-                       for (std::set<std::string>::iterator cs = contents_s.begin();
-                                       cs != contents_s.end(); ++cs) {
+                       for (const std::string &content_s : contents_s) {
                                std::set<content_t> ids;
-                               ndef->getIds(*cs, ids);
-                               for (std::set<content_t>::const_iterator k = ids.begin();
-                                               k != ids.end(); ++k) {
-                                       content_t c = *k;
+                               ndef->getIds(content_s, ids);
+                               for (content_t c : ids) {
                                        if (c >= m_aabms.size())
                                                m_aabms.resize(c + 256, NULL);
                                        if (!m_aabms[c])
@@ -762,8 +718,8 @@ class ABMHandler
 
        ~ABMHandler()
        {
-               for (size_t i = 0; i < m_aabms.size(); i++)
-                       delete m_aabms[i];
+               for (auto &aabms : m_aabms)
+                       delete aabms;
        }
 
        // Find out how many objects the given block and its neighbours contain.
@@ -817,14 +773,12 @@ class ABMHandler
                                continue;
 
                        v3s16 p = p0 + block->getPosRelative();
-                       for(std::vector<ActiveABM>::iterator
-                               i = m_aabms[c]->begin(); i != m_aabms[c]->end(); ++i) {
-                               if(myrand() % i->chance != 0)
+                       for (ActiveABM &aabm : *m_aabms[c]) {
+                               if (myrand() % aabm.chance != 0)
                                        continue;
 
                                // Check neighbors
-                               if(!i->required_neighbors.empty())
-                               {
+                               if (!aabm.required_neighbors.empty()) {
                                        v3s16 p1;
                                        for(p1.X = p0.X-1; p1.X <= p0.X+1; p1.X++)
                                        for(p1.Y = p0.Y-1; p1.Y <= p0.Y+1; p1.Y++)
@@ -844,8 +798,8 @@ class ABMHandler
                                                        c = n.getContent();
                                                }
                                                std::set<content_t>::const_iterator k;
-                                               k = i->required_neighbors.find(c);
-                                               if(k != i->required_neighbors.end()){
+                                               k = aabm.required_neighbors.find(c);
+                                               if(k != aabm.required_neighbors.end()){
                                                        goto neighbor_found;
                                                }
                                        }
@@ -855,8 +809,8 @@ class ABMHandler
                                neighbor_found:
 
                                // Call all the trigger variations
-                               i->abm->trigger(m_env, p, n);
-                               i->abm->trigger(m_env, p, n,
+                               aabm.abm->trigger(m_env, p, n);
+                               aabm.abm->trigger(m_env, p, n,
                                        active_object_count, active_object_count_wider);
 
                                // Count surrounding objects again if the abms added any
@@ -912,13 +866,12 @@ void ServerEnvironment::activateBlock(MapBlock *block, u32 additional_dtime)
                block->m_node_timers.step((float)dtime_s);
        if (!elapsed_timers.empty()) {
                MapNode n;
-               for (std::vector<NodeTimer>::iterator
-                       i = elapsed_timers.begin();
-                       i != elapsed_timers.end(); ++i){
-                       n = block->getNodeNoEx(i->position);
-                       v3s16 p = i->position + block->getPosRelative();
-                       if (m_script->node_on_timer(p, n, i->elapsed))
-                               block->setNodeTimer(NodeTimer(i->timeout, 0, i->position));
+               for (const NodeTimer &elapsed_timer : elapsed_timers) {
+                       n = block->getNodeNoEx(elapsed_timer.position);
+                       v3s16 p = elapsed_timer.position + block->getPosRelative();
+                       if (m_script->node_on_timer(p, n, elapsed_timer.elapsed))
+                               block->setNodeTimer(NodeTimer(elapsed_timer.timeout, 0,
+                                       elapsed_timer.position));
                }
        }
 
@@ -929,7 +882,7 @@ void ServerEnvironment::activateBlock(MapBlock *block, u32 additional_dtime)
 
 void ServerEnvironment::addActiveBlockModifier(ActiveBlockModifier *abm)
 {
-       m_abms.push_back(ABMWithState(abm));
+       m_abms.emplace_back(abm);
 }
 
 void ServerEnvironment::addLoadingBlockModifierDef(LoadingBlockModifierDef *lbm)
@@ -1003,10 +956,9 @@ bool ServerEnvironment::swapNode(v3s16 p, const MapNode &n)
 void ServerEnvironment::getObjectsInsideRadius(std::vector<u16> &objects, v3f pos,
        float radius)
 {
-       for (ServerActiveObjectMap::iterator i = m_active_objects.begin();
-               i != m_active_objects.end(); ++i) {
-               ServerActiveObject* obj = i->second;
-               u16 id = i->first;
+       for (auto &activeObject : m_active_objects) {
+               ServerActiveObject* obj = activeObject.second;
+               u16 id = activeObject.first;
                v3f objectpos = obj->getBasePosition();
                if (objectpos.getDistanceFrom(pos) > radius)
                        continue;
@@ -1019,12 +971,11 @@ void ServerEnvironment::clearObjects(ClearObjectsMode mode)
        infostream << "ServerEnvironment::clearObjects(): "
                << "Removing all active objects" << std::endl;
        std::vector<u16> objects_to_remove;
-       for (ServerActiveObjectMap::iterator i = m_active_objects.begin();
-               i != m_active_objects.end(); ++i) {
-               ServerActiveObject* obj = i->second;
+       for (auto &it : m_active_objects) {
+               ServerActiveObject* obj = it.second;
                if (obj->getType() == ACTIVEOBJECT_TYPE_PLAYER)
                        continue;
-               u16 id = i->first;
+               u16 id = it.first;
                // Delete static object if block is loaded
                if (obj->m_static_exists) {
                        MapBlock *block = m_map->getBlockNoCreateNoEx(obj->m_static_block);
@@ -1055,9 +1006,8 @@ void ServerEnvironment::clearObjects(ClearObjectsMode mode)
        }
 
        // Remove references from m_active_objects
-       for (std::vector<u16>::iterator i = objects_to_remove.begin();
-               i != objects_to_remove.end(); ++i) {
-               m_active_objects.erase(*i);
+       for (u16 i : objects_to_remove) {
+               m_active_objects.erase(i);
        }
 
        // Get list of loaded blocks
@@ -1087,9 +1037,7 @@ void ServerEnvironment::clearObjects(ClearObjectsMode mode)
                << " blocks" << std::endl;
 
        // Grab a reference on each loaded block to avoid unloading it
-       for (std::vector<v3s16>::iterator i = loaded_blocks.begin();
-               i != loaded_blocks.end(); ++i) {
-               v3s16 p = *i;
+       for (v3s16 p : loaded_blocks) {
                MapBlock *block = m_map->getBlockNoCreateNoEx(p);
                assert(block != NULL);
                block->refGrab();
@@ -1105,7 +1053,7 @@ void ServerEnvironment::clearObjects(ClearObjectsMode mode)
        u32 num_blocks_checked = 0;
        u32 num_blocks_cleared = 0;
        u32 num_objs_cleared = 0;
-       for (std::vector<v3s16>::iterator i = loadable_blocks.begin();
+       for (auto i = loadable_blocks.begin();
                i != loadable_blocks.end(); ++i) {
                v3s16 p = *i;
                MapBlock *block = m_map->emergeBlock(p, false);
@@ -1142,9 +1090,7 @@ void ServerEnvironment::clearObjects(ClearObjectsMode mode)
        m_map->unloadUnreferencedBlocks();
 
        // Drop references that were added above
-       for (std::vector<v3s16>::iterator i = loaded_blocks.begin();
-               i != loaded_blocks.end(); ++i) {
-               v3s16 p = *i;
+       for (v3s16 p : loaded_blocks) {
                MapBlock *block = m_map->getBlockNoCreateNoEx(p);
                assert(block);
                block->refDrop();
@@ -1188,17 +1134,13 @@ void ServerEnvironment::step(float dtime)
        */
        {
                ScopeProfiler sp(g_profiler, "SEnv: handle players avg", SPT_AVG);
-               for (std::vector<RemotePlayer *>::iterator i = m_players.begin();
-                       i != m_players.end(); ++i) {
-                       RemotePlayer *player = dynamic_cast<RemotePlayer *>(*i);
-                       assert(player);
-
+               for (RemotePlayer *player : m_players) {
                        // Ignore disconnected players
-                       if(player->peer_id == 0)
+                       if (player->peer_id == 0)
                                continue;
 
                        // Move
-                       player->move(dtime, this, 100*BS);
+                       player->move(dtime, this, 100 * BS);
                }
        }
 
@@ -1211,11 +1153,7 @@ void ServerEnvironment::step(float dtime)
                        Get player block positions
                */
                std::vector<v3s16> players_blockpos;
-               for (std::vector<RemotePlayer *>::iterator i = m_players.begin();
-                       i != m_players.end(); ++i) {
-                       RemotePlayer *player = dynamic_cast<RemotePlayer *>(*i);
-                       assert(player);
-
+               for (RemotePlayer *player: m_players) {
                        // Ignore disconnected players
                        if (player->peer_id == 0)
                                continue;
@@ -1223,9 +1161,8 @@ void ServerEnvironment::step(float dtime)
                        PlayerSAO *playersao = player->getPlayerSAO();
                        assert(playersao);
 
-                       v3s16 blockpos = getNodeBlockPos(
-                               floatToInt(playersao->getBasePosition(), BS));
-                       players_blockpos.push_back(blockpos);
+                       players_blockpos.push_back(
+                               getNodeBlockPos(floatToInt(playersao->getBasePosition(), BS)));
                }
 
                /*
@@ -1245,16 +1182,9 @@ void ServerEnvironment::step(float dtime)
                // Convert active objects that are no more in active blocks to static
                deactivateFarObjects(false);
 
-               for(std::set<v3s16>::iterator
-                       i = blocks_removed.begin();
-                       i != blocks_removed.end(); ++i) {
-                       v3s16 p = *i;
-
-                       /* infostream<<"Server: Block " << PP(p)
-                               << " became inactive"<<std::endl; */
-
+               for (const v3s16 &p: blocks_removed) {
                        MapBlock *block = m_map->getBlockNoCreateNoEx(p);
-                       if(block==NULL)
+                       if (!block)
                                continue;
 
                        // Set current time as timestamp (and let it set ChangedFlag)
@@ -1265,21 +1195,14 @@ void ServerEnvironment::step(float dtime)
                        Handle added blocks
                */
 
-               for(std::set<v3s16>::iterator
-                       i = blocks_added.begin();
-                       i != blocks_added.end(); ++i)
-               {
-                       v3s16 p = *i;
-
+               for (const v3s16 &p: blocks_added) {
                        MapBlock *block = m_map->getBlockOrEmerge(p);
-                       if(block==NULL){
+                       if (!block) {
                                m_active_blocks.m_list.erase(p);
                                continue;
                        }
 
                        activateBlock(block);
-                       /* infostream<<"Server: Block " << PP(p)
-                               << " became active"<<std::endl; */
                }
        }
 
@@ -1291,17 +1214,9 @@ void ServerEnvironment::step(float dtime)
 
                float dtime = m_cache_nodetimer_interval;
 
-               for(std::set<v3s16>::iterator
-                       i = m_active_blocks.m_list.begin();
-                       i != m_active_blocks.m_list.end(); ++i)
-               {
-                       v3s16 p = *i;
-
-                       /*infostream<<"Server: Block ("<<p.X<<","<<p.Y<<","<<p.Z
-                                       <<") being handled"<<std::endl;*/
-
+               for (const v3s16 &p: m_active_blocks.m_list) {
                        MapBlock *block = m_map->getBlockNoCreateNoEx(p);
-                       if(block==NULL)
+                       if (!block)
                                continue;
 
                        // Reset block usage timer
@@ -1316,17 +1231,16 @@ void ServerEnvironment::step(float dtime)
                                        MOD_REASON_BLOCK_EXPIRED);
 
                        // Run node timers
-                       std::vector<NodeTimer> elapsed_timers =
-                               block->m_node_timers.step((float)dtime);
+                       std::vector<NodeTimer> elapsed_timers = block->m_node_timers.step(dtime);
                        if (!elapsed_timers.empty()) {
                                MapNode n;
-                               for (std::vector<NodeTimer>::iterator i = elapsed_timers.begin();
-                                       i != elapsed_timers.end(); ++i) {
-                                       n = block->getNodeNoEx(i->position);
-                                       p = i->position + block->getPosRelative();
-                                       if (m_script->node_on_timer(p, n, i->elapsed)) {
+                               v3s16 p2;
+                               for (const NodeTimer &elapsed_timer: elapsed_timers) {
+                                       n = block->getNodeNoEx(elapsed_timer.position);
+                                       p2 = elapsed_timer.position + block->getPosRelative();
+                                       if (m_script->node_on_timer(p2, n, elapsed_timer.elapsed)) {
                                                block->setNodeTimer(NodeTimer(
-                                                       i->timeout, 0, i->position));
+                                                       elapsed_timer.timeout, 0, elapsed_timer.position));
                                        }
                                }
                        }
@@ -1334,8 +1248,8 @@ void ServerEnvironment::step(float dtime)
        }
 
        if (m_active_block_modifier_interval.step(dtime, m_cache_abm_interval))
-               do{ // breakable
-                       if(m_active_block_interval_overload_skip > 0){
+               do { // breakable
+                       if (m_active_block_interval_overload_skip > 0) {
                                ScopeProfiler sp(g_profiler, "SEnv: ABM overload skips");
                                m_active_block_interval_overload_skip--;
                                break;
@@ -1346,17 +1260,9 @@ void ServerEnvironment::step(float dtime)
                        // Initialize handling of ActiveBlockModifiers
                        ABMHandler abmhandler(m_abms, m_cache_abm_interval, this, true);
 
-                       for(std::set<v3s16>::iterator
-                               i = m_active_blocks.m_list.begin();
-                               i != m_active_blocks.m_list.end(); ++i)
-                       {
-                               v3s16 p = *i;
-
-                               /*infostream<<"Server: Block ("<<p.X<<","<<p.Y<<","<<p.Z
-                                               <<") being handled"<<std::endl;*/
-
+                       for (const v3s16 &p : m_active_blocks.m_list) {
                                MapBlock *block = m_map->getBlockNoCreateNoEx(p);
-                               if(block == NULL)
+                               if (!block)
                                        continue;
 
                                // Set current time as timestamp
@@ -1368,7 +1274,7 @@ void ServerEnvironment::step(float dtime)
 
                        u32 time_ms = timer.stop(true);
                        u32 max_time_ms = 200;
-                       if(time_ms > max_time_ms){
+                       if (time_ms > max_time_ms) {
                                warningstream<<"active block modifiers took "
                                        <<time_ms<<"ms (longer than "
                                        <<max_time_ms<<"ms)"<<std::endl;
@@ -1399,9 +1305,8 @@ void ServerEnvironment::step(float dtime)
                        send_recommended = true;
                }
 
-               for (ServerActiveObjectMap::iterator i = m_active_objects.begin();
-                       i != m_active_objects.end(); ++i) {
-                       ServerActiveObject* obj = i->second;
+               for (auto &ao_it : m_active_objects) {
+                       ServerActiveObject* obj = ao_it.second;
                        // Don't step if is to be removed or stored statically
                        if(obj->m_removed || obj->m_pending_deactivation)
                                continue;
@@ -1420,8 +1325,7 @@ void ServerEnvironment::step(float dtime)
        /*
                Manage active objects
        */
-       if(m_object_management_interval.step(dtime, 0.5))
-       {
+       if (m_object_management_interval.step(dtime, 0.5)) {
                ScopeProfiler sp(g_profiler, "SEnv: remove removed objs avg /.5s", SPT_AVG);
                /*
                        Remove objects that satisfy (m_removed && m_known_by_count==0)
@@ -1480,9 +1384,9 @@ u32 ServerEnvironment::addParticleSpawner(float exptime, u16 attached_id)
 void ServerEnvironment::deleteParticleSpawner(u32 id, bool remove_from_object)
 {
        m_particle_spawners.erase(id);
-       std::unordered_map<u32, u16>::iterator it = m_particle_spawner_attachments.find(id);
+       const auto &it = m_particle_spawner_attachments.find(id);
        if (it != m_particle_spawner_attachments.end()) {
-               u16 obj_id = (*it).second;
+               u16 obj_id = it->second;
                ServerActiveObject *sao = getActiveObject(obj_id);
                if (sao != NULL && remove_from_object) {
                        sao->detachParticleSpawner(id);
@@ -1550,12 +1454,11 @@ void ServerEnvironment::getAddedActiveObjects(PlayerSAO *playersao, s16 radius,
                - discard objects that are found in current_objects.
                - add remaining objects to added_objects
        */
-       for (ServerActiveObjectMap::iterator i = m_active_objects.begin();
-               i != m_active_objects.end(); ++i) {
-               u16 id = i->first;
+       for (auto &ao_it : m_active_objects) {
+               u16 id = ao_it.first;
 
                // Get object
-               ServerActiveObject *object = i->second;
+               ServerActiveObject *object = ao_it.second;
                if (object == NULL)
                        continue;
 
@@ -1604,11 +1507,7 @@ void ServerEnvironment::getRemovedActiveObjects(PlayerSAO *playersao, s16 radius
                - object has m_removed=true, or
                - object is too far away
        */
-       for(std::set<u16>::iterator
-               i = current_objects.begin();
-               i != current_objects.end(); ++i)
-       {
-               u16 id = *i;
+       for (u16 id : current_objects) {
                ServerActiveObject *object = getActiveObject(id);
 
                if (object == NULL) {
@@ -1642,11 +1541,9 @@ void ServerEnvironment::setStaticForActiveObjectsInBlock(
        if (!block)
                return;
 
-       for (std::map<u16, StaticObject>::iterator
-               so_it = block->m_static_objects.m_active.begin();
-               so_it != block->m_static_objects.m_active.end(); ++so_it) {
+       for (auto &so_it : block->m_static_objects.m_active) {
                // Get the ServerActiveObject counterpart to this StaticObject
-               ServerActiveObjectMap::const_iterator ao_it = m_active_objects.find(so_it->first);
+               ServerActiveObjectMap::const_iterator ao_it = m_active_objects.find(so_it.first);
                if (ao_it == m_active_objects.end()) {
                        // If this ever happens, there must be some kind of nasty bug.
                        errorstream << "ServerEnvironment::setStaticForObjectsInBlock(): "
@@ -1671,6 +1568,38 @@ ActiveObjectMessage ServerEnvironment::getActiveObjectMessage()
        return message;
 }
 
+void ServerEnvironment::getSelectedActiveObjects(
+       const core::line3d<f32> &shootline_on_map,
+       std::vector<PointedThing> &objects)
+{
+       std::vector<u16> objectIds;
+       getObjectsInsideRadius(objectIds, shootline_on_map.start,
+               shootline_on_map.getLength() + 10.0f);
+       const v3f line_vector = shootline_on_map.getVector();
+
+       for (u16 objectId : objectIds) {
+               ServerActiveObject* obj = getActiveObject(objectId);
+
+               aabb3f selection_box;
+               if (!obj->getSelectionBox(&selection_box))
+                       continue;
+
+               v3f pos = obj->getBasePosition();
+
+               aabb3f offsetted_box(selection_box.MinEdge + pos,
+                       selection_box.MaxEdge + pos);
+
+               v3f current_intersection;
+               v3s16 current_normal;
+               if (boxLineCollision(offsetted_box, shootline_on_map.start, line_vector,
+                               &current_intersection, &current_normal)) {
+                       objects.emplace_back(
+                               (s16) objectId, current_intersection, current_normal,
+                               (current_intersection - shootline_on_map.start).getLengthSQ());
+               }
+       }
+}
+
 /*
        ************ Private methods *************
 */
@@ -1734,7 +1663,7 @@ u16 ServerEnvironment::addActiveObjectRaw(ServerActiveObject *object,
        {
                // Add static object to active static list of the block
                v3f objectpos = object->getBasePosition();
-               std::string staticdata = "";
+               std::string staticdata;
                object->getStaticData(&staticdata);
                StaticObject s_obj(object->getType(), objectpos, staticdata);
                // Add to the block where the object is located in
@@ -1765,10 +1694,9 @@ u16 ServerEnvironment::addActiveObjectRaw(ServerActiveObject *object,
 void ServerEnvironment::removeRemovedObjects()
 {
        std::vector<u16> objects_to_remove;
-       for(ServerActiveObjectMap::iterator i = m_active_objects.begin();
-               i != m_active_objects.end(); ++i) {
-               u16 id = i->first;
-               ServerActiveObject* obj = i->second;
+       for (auto &ao_it : m_active_objects) {
+               u16 id = ao_it.first;
+               ServerActiveObject* obj = ao_it.second;
                // This shouldn't happen but check it
                if(obj == NULL)
                {
@@ -1841,9 +1769,8 @@ void ServerEnvironment::removeRemovedObjects()
                objects_to_remove.push_back(id);
        }
        // Remove references from m_active_objects
-       for(std::vector<u16>::iterator i = objects_to_remove.begin();
-               i != objects_to_remove.end(); ++i) {
-               m_active_objects.erase(*i);
+       for (u16 i : objects_to_remove) {
+               m_active_objects.erase(i);
        }
 }
 
@@ -1912,11 +1839,7 @@ void ServerEnvironment::activateObjects(MapBlock *block, u32 dtime_s)
 
        // Activate stored objects
        std::vector<StaticObject> new_stored;
-       for (std::vector<StaticObject>::iterator
-               i = block->m_static_objects.m_stored.begin();
-               i != block->m_static_objects.m_stored.end(); ++i) {
-               StaticObject &s_obj = *i;
-
+       for (const StaticObject &s_obj : block->m_static_objects.m_stored) {
                // Create an active object from the data
                ServerActiveObject *obj = ServerActiveObject::create
                        ((ActiveObjectType) s_obj.type, this, 0, s_obj.pos, s_obj.data);
@@ -1940,20 +1863,14 @@ void ServerEnvironment::activateObjects(MapBlock *block, u32 dtime_s)
        // Clear stored list
        block->m_static_objects.m_stored.clear();
        // Add leftover failed stuff to stored list
-       for(std::vector<StaticObject>::iterator
-               i = new_stored.begin();
-               i != new_stored.end(); ++i) {
-               StaticObject &s_obj = *i;
+       for (const StaticObject &s_obj : new_stored) {
                block->m_static_objects.m_stored.push_back(s_obj);
        }
 
        // Turn the active counterparts of activated objects not pending for
        // deactivation
-       for(std::map<u16, StaticObject>::iterator
-               i = block->m_static_objects.m_active.begin();
-               i != block->m_static_objects.m_active.end(); ++i)
-       {
-               u16 id = i->first;
+       for (auto &i : block->m_static_objects.m_active) {
+               u16 id = i.first;
                ServerActiveObject *object = getActiveObject(id);
                assert(object);
                object->m_pending_deactivation = false;
@@ -1983,12 +1900,11 @@ void ServerEnvironment::activateObjects(MapBlock *block, u32 dtime_s)
 void ServerEnvironment::deactivateFarObjects(bool _force_delete)
 {
        std::vector<u16> objects_to_remove;
-       for (ServerActiveObjectMap::iterator i = m_active_objects.begin();
-               i != m_active_objects.end(); ++i) {
+       for (auto &ao_it : m_active_objects) {
                // force_delete might be overriden per object
                bool force_delete = _force_delete;
 
-               ServerActiveObject* obj = i->second;
+               ServerActiveObject* obj = ao_it.second;
                assert(obj);
 
                // Do not deactivate if static data creation not allowed
@@ -1999,7 +1915,7 @@ void ServerEnvironment::deactivateFarObjects(bool _force_delete)
                if(!force_delete && obj->m_pending_deactivation)
                        continue;
 
-               u16 id = i->first;
+               u16 id = ao_it.first;
                v3f objectpos = obj->getBasePosition();
 
                // The block in which the object resides in
@@ -2024,7 +1940,7 @@ void ServerEnvironment::deactivateFarObjects(bool _force_delete)
                                        <<std::endl;
                                continue;
                        }
-                       std::string staticdata_new = "";
+                       std::string staticdata_new;
                        obj->getStaticData(&staticdata_new);
                        StaticObject s_obj(obj->getType(), objectpos, staticdata_new);
                        block->m_static_objects.insert(id, s_obj);
@@ -2065,7 +1981,7 @@ void ServerEnvironment::deactivateFarObjects(bool _force_delete)
                if(obj->isStaticAllowed())
                {
                        // Create new static object
-                       std::string staticdata_new = "";
+                       std::string staticdata_new;
                        obj->getStaticData(&staticdata_new);
                        StaticObject s_obj(obj->getType(), objectpos, staticdata_new);
 
@@ -2206,9 +2122,8 @@ void ServerEnvironment::deactivateFarObjects(bool _force_delete)
        }
 
        // Remove references from m_active_objects
-       for(std::vector<u16>::iterator i = objects_to_remove.begin();
-               i != objects_to_remove.end(); ++i) {
-               m_active_objects.erase(*i);
+       for (u16 i : objects_to_remove) {
+               m_active_objects.erase(i);
        }
 }
 
@@ -2218,19 +2133,20 @@ PlayerDatabase *ServerEnvironment::openPlayerDatabase(const std::string &name,
 
        if (name == "sqlite3")
                return new PlayerDatabaseSQLite3(savedir);
-       else if (name == "dummy")
+
+       if (name == "dummy")
                return new Database_Dummy();
 #if USE_POSTGRESQL
-       else if (name == "postgresql") {
-               std::string connect_string = "";
+       if (name == "postgresql") {
+               std::string connect_string;
                conf.getNoEx("pgsql_player_connection", connect_string);
                return new PlayerDatabasePostgreSQL(connect_string);
        }
 #endif
-       else if (name == "files")
+       if (name == "files")
                return new PlayerDatabaseFiles(savedir + DIR_DELIM + "players");
-       else
-               throw BaseException(std::string("Database backend ") + name + " not supported.");
+
+       throw BaseException(std::string("Database backend ") + name + " not supported.");
 }
 
 bool ServerEnvironment::migratePlayersDatabase(const GameParams &game_params,