Add online content repository

[dragonfireclient.git] / src / server / serveractiveobjectmap.cpp

diff --git a/src/server/serveractiveobjectmap.cpp b/src/server/serveractiveobjectmap.cpp
index e2def776dd1553478ec5d1c2f5b99cd83768cf06..7c5c1c1ea212e107db9372227100b18bf9b7bc3b 100644 (file)
--- a/src/server/serveractiveobjectmap.cpp
+++ b/src/server/serveractiveobjectmap.cpp
@@ -18,21 +18,24 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 */
 
 #include "serveractiveobjectmap.h"
+#include <cmath>
 #include "constants.h"
 #include "log.h"
 #include "serverobject.h"
 
 static constexpr float granularity = 16.0 * BS;
 
+static v3s16 getChunkPos(const v3f &pos)
+{
+       return v3s16(
+                       std::floor(pos.X / granularity),
+                       std::floor(pos.Y / granularity),
+                       std::floor(pos.Z / granularity));
+}
+
 static aabb3s16 calcBox(const aabb3f &cb)
 {
-       return aabb3s16(
-                       floor(cb.MinEdge.X / granularity),
-                       floor(cb.MinEdge.Y / granularity),
-                       floor(cb.MinEdge.Z / granularity),
-                       ceil(cb.MaxEdge.X / granularity),
-                       ceil(cb.MaxEdge.Y / granularity),
-                       ceil(cb.MaxEdge.Z / granularity));
+       return { getChunkPos(cb.MinEdge), getChunkPos(cb.MaxEdge) };
 }
 
 void ServerActiveObjectMap::addObject(ServerActiveObject *object)
@@ -45,10 +48,12 @@ void ServerActiveObjectMap::addObject(ServerActiveObject *object)
                        "object ID in use: " + std::to_string(id));
        w.object = object;
        w.has_box = w.object->getCollisionBox(&cb);
+       w.pos = getChunkPos(w.object->getBasePosition());
        if (w.has_box) {
                w.box = calcBox(cb);
                addObjectRefs(id, w.box);
        }
+       addObjectRef(id, w.pos);
        objects.emplace(id, w);
 }
 
@@ -60,6 +65,7 @@ ServerActiveObject *ServerActiveObjectMap::removeObject(u16 id)
        Wrapper w = pw->second;
        if (w.has_box)
                removeObjectRefs(id, w.box);
+       removeObjectRef(id, w.pos);
        objects.erase(pw);
        return w.object;
 }
@@ -78,19 +84,22 @@ void ServerActiveObjectMap::updateObject(u16 id)
                return;
        }
        Wrapper &w = pw->second;
+       v3s16 pos = getChunkPos(w.object->getBasePosition());
        aabb3f cb;
        aabb3s16 box;
        bool has_box = w.object->getCollisionBox(&cb);
        if (has_box)
                box = calcBox(cb);
-       if (w.has_box && has_box && w.box == box)
+       if (w.has_box && has_box && w.box == box && pos == w.pos)
                return;
        if (w.has_box)
                removeObjectRefs(id, w.box);
+       removeObjectRef(id, w.pos);
        w.box = box;
        w.has_box = has_box;
        if (w.has_box)
                addObjectRefs(id, w.box);
+       addObjectRef(id, w.pos);
 }
 
 void ServerActiveObjectMap::updateObject(ServerActiveObject *object)
@@ -150,13 +159,29 @@ std::unordered_set<u16> ServerActiveObjectMap::getObjectsNearBox(const aabb3s16
        return result;
 }
 
+void ServerActiveObjectMap::addObjectRef(u16 id, v3s16 pos)
+{
+       refmap.emplace(pos, id);
+}
+
+void ServerActiveObjectMap::removeObjectRef(u16 id, v3s16 pos)
+{
+       auto bounds = refmap.equal_range(pos);
+       for (auto iter = bounds.first; iter != bounds.second;) {
+               if (iter->second == id)
+                       iter = refmap.erase(iter);
+               else
+                       ++iter;
+       }
+}
+
 void ServerActiveObjectMap::addObjectRefs(u16 id, const aabb3s16 &box)
 {
        v3s16 p;
        for (p.Z = box.MinEdge.Z; p.Z <= box.MaxEdge.Z; p.Z++)
        for (p.Y = box.MinEdge.Y; p.Y <= box.MaxEdge.Y; p.Y++)
        for (p.X = box.MinEdge.X; p.X <= box.MaxEdge.X; p.X++)
-               refmap.emplace(p, id);
+               addObjectRef(id, p);
 }
 
 void ServerActiveObjectMap::removeObjectRefs(u16 id, const aabb3s16 &box)
@@ -164,14 +189,8 @@ void ServerActiveObjectMap::removeObjectRefs(u16 id, const aabb3s16 &box)
        v3s16 p;
        for (p.Z = box.MinEdge.Z; p.Z <= box.MaxEdge.Z; p.Z++)
        for (p.Y = box.MinEdge.Y; p.Y <= box.MaxEdge.Y; p.Y++)
-       for (p.X = box.MinEdge.X; p.X <= box.MaxEdge.X; p.X++) {
-               auto bounds = refmap.equal_range(p);
-               for (auto iter = bounds.first; iter != bounds.second;)
-                       if (iter->second == id)
-                               refmap.erase(iter++);
-                       else
-                               ++iter;
-       }
+       for (p.X = box.MinEdge.X; p.X <= box.MaxEdge.X; p.X++)
+               removeObjectRef(id, p);
 }
 
 bool ServerActiveObjectMap::isFreeId(u16 id)