diff options
author | Yohann Ferreira <yohann_dot_ferreira_at_orange_dot_efer> | 2011-08-26 06:18:31 +0800 |
---|---|---|
committer | Yohann Ferreira <yohann.ferreira@orange.fr> | 2011-09-28 22:36:43 +0800 |
commit | da782617397773fb2f29a4021fe73a80e38f62dd (patch) | |
tree | 10d13f9e357141c237c6d22760479fb8a7867d41 /src | |
parent | 7ddeb3c927b81474548bbc13ebcd00eb5688c51a (diff) | |
download | manaserv-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.h | 3 | ||||
-rw-r--r-- | src/game-server/trigger.cpp | 5 |
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 |