From f07dbf97c27aef7630782052ba6021359294cd86 Mon Sep 17 00:00:00 2001
From: Guillaume Melquiond <guillaume.melquiond@gmail.com>
Date: Sun, 1 Jul 2007 18:06:25 +0000
Subject: Added assertions to detect insertion and removal of objects at
 updating time.

---
 src/game-server/state.cpp | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

(limited to 'src')

diff --git a/src/game-server/state.cpp b/src/game-server/state.cpp
index c27f9209..645906c1 100644
--- a/src/game-server/state.cpp
+++ b/src/game-server/state.cpp
@@ -277,8 +277,16 @@ void State::informPlayer(MapComposite *map, Character *p)
         gameHandler->sendTo(p, itemMsg);
 }
 
+#ifndef NDEBUG
+static bool dbgLockObjects;
+#endif
+
 void State::update()
 {
+#   ifndef NDEBUG
+    dbgLockObjects = true;
+#   endif
+
     // Update game state (update AI, etc.)
     for (Maps::iterator m = maps.begin(), m_end = maps.end(); m != m_end; ++m)
     {
@@ -301,6 +309,10 @@ void State::update()
         }
     }
 
+#   ifndef NDEBUG
+    dbgLockObjects = false;
+#   endif
+
     // Take care of events that were delayed because of their side effects.
     for (DelayedEvents::iterator i = delayedEvents.begin(),
          i_end = delayedEvents.end(); i != i_end; ++i)
@@ -359,6 +371,7 @@ void State::update()
 
 void State::insert(Thing *ptr)
 {
+    assert(!dbgLockObjects);
     int mapId = ptr->getMapId();
     MapComposite *map = loadMap(mapId);
     if (!map || !map->insert(ptr))
@@ -386,6 +399,7 @@ void State::insert(Thing *ptr)
 
 void State::remove(Thing *ptr)
 {
+    assert(!dbgLockObjects);
     int mapId = ptr->getMapId();
     Maps::iterator m = maps.find(mapId);
     assert(m != maps.end());
-- 
cgit v1.2.3-60-g2f50