summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorYohann Ferreira <yohann_dot_ferreira_at_orange_dot_efer>2011-08-26 06:18:31 +0800
committerYohann Ferreira <yohann.ferreira@orange.fr>2011-09-28 22:36:43 +0800
commitda782617397773fb2f29a4021fe73a80e38f62dd (patch)
tree10d13f9e357141c237c6d22760479fb8a7867d41 /src
parent7ddeb3c927b81474548bbc13ebcd00eb5688c51a (diff)
downloadmanaserv-da782617397773fb2f29a4021fe73a80e38f62dd.tar.gz
manaserv-da782617397773fb2f29a4021fe73a80e38f62dd.tar.bz2
manaserv-da782617397773fb2f29a4021fe73a80e38f62dd.tar.xz
manaserv-da782617397773fb2f29a4021fe73a80e38f62dd.zip
Added a bit more precision on the conditions used in the TriggerArea::update() function.
Diffstat (limited to 'src')
-rw-r--r--src/game-server/actor.h3
-rw-r--r--src/game-server/trigger.cpp5
2 files changed, 6 insertions, 2 deletions
diff --git a/src/game-server/actor.h b/src/game-server/actor.h
index 529d48c4..abc1e1d0 100644
--- a/src/game-server/actor.h
+++ b/src/game-server/actor.h
@@ -111,6 +111,9 @@ class Actor : public Thing
void setPublicID(int id)
{ mPublicID = id; }
+ bool isPublicIdValid() const
+ { return (mPublicID > 0 && mPublicID != 65535); }
+
/**
* Gets the way the actor blocks pathfinding for other actors.
*/
diff --git a/src/game-server/trigger.cpp b/src/game-server/trigger.cpp
index 19ca3512..c4cec2f4 100644
--- a/src/game-server/trigger.cpp
+++ b/src/game-server/trigger.cpp
@@ -52,8 +52,9 @@ void TriggerArea::update()
std::set<Actor*> insideNow;
for (BeingIterator i(getMap()->getInsideRectangleIterator(mZone)); i; ++i)
{
- //skip garbage
- if (!(*i) || (*i)->getPublicID() == 0) continue;
+ // Don't deal with unitialized actors.
+ if (!(*i) || !(*i)->isPublicIdValid())
+ continue;
// The BeingIterator returns the mapZones in touch with the rectangle
// area. On the other hand, the beings contained in the map zones