diff options
author | Loïc Blot <nerzhul@users.noreply.github.com> | 2020-04-16 08:25:48 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-04-16 08:25:48 +0200 |
commit | e8ac5a31cf12afcfddf8e3ed31e8038930edb06f (patch) | |
tree | 5bdc776c5bbe190e32e42899b455e6d16179e3b5 /src/server/activeobjectmgr.h | |
parent | 62ae7adab2bebde04864c12543caefbffab24963 (diff) | |
download | hax-minetest-server-e8ac5a31cf12afcfddf8e3ed31e8038930edb06f.tar.gz hax-minetest-server-e8ac5a31cf12afcfddf8e3ed31e8038930edb06f.zip |
Optimize get_objects_inside_radius calls (#9671)
* Optimize getObjectsInsideRadius calls
our previous implementation calls the ActiveObjectMgr to return ids and then lookup those ids in the same map and test each object
Instead now we call the global map to return the pointers directly and we ask filtering when building the list using lamba.
This drop double looping over ranges of active objects (and then filtered one) and drop x lookups on the map regarding the first call results
Diffstat (limited to 'src/server/activeobjectmgr.h')
-rw-r--r-- | src/server/activeobjectmgr.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/server/activeobjectmgr.h b/src/server/activeobjectmgr.h index 5fea1bea6..bc2085499 100644 --- a/src/server/activeobjectmgr.h +++ b/src/server/activeobjectmgr.h @@ -35,8 +35,9 @@ public: bool registerObject(ServerActiveObject *obj) override; void removeObject(u16 id) override; - void getObjectsInsideRadius( - const v3f &pos, float radius, std::vector<u16> &result); + void getObjectsInsideRadius(const v3f &pos, float radius, + std::vector<ServerActiveObject *> &result, + std::function<bool(ServerActiveObject *obj)> include_obj_cb); void getAddedActiveObjectsAroundPos(const v3f &player_pos, f32 radius, f32 player_radius, std::set<u16> ¤t_objects, |