Changed Enemies to look into movement direction.

source/abstract/Character.cpp

@@ -17,6 +17,7 @@
 #include "../Pathfinder.h"
 
 const float Character::VISION_DISTANCE = 500.0f;
+const float Character::POINT_REACHED_DISTANCE = 1.0f;
 
 /**
  * Saves pointer to this instance in static var for think().
@@ -118,17 +119,7 @@ Character::releaseTrigger() {
  */
 bool
 Character::setDestination(const sf::Vector2f& destination) {
-	if (destination == getPosition()) {
-		mPath.clear();
-		return true;
-	}
 	mPath = mPathfinder.getPath(getPosition(), destination, getRadius());
-	if (!mPath.empty())
-		setSpeed(mPath.back() - getPosition(), mMovementSpeed);
-	else {
-		setSpeed(sf::Vector2f(), 0);
-		LOG_W("No path found to destination.");
-	}
 	return !mPath.empty();
 }
 
@@ -138,16 +129,15 @@ Character::setDestination(const sf::Vector2f& destination) {
  */
 void
 Character::move() {
-	sf::Vector2f distanceTraveled = mLastPosition - getPosition();
+	if (mPath.empty())
+		return;
 	mLastPosition = getPosition();
-	if (!mPath.empty()) {
+	setSpeed(mPath.back() - getPosition(), mMovementSpeed);
-		// Point reached (during next time step).
+	setDirection(mPath.back() - getPosition());
-		if (thor::length(mPath.back() - getPosition()) < thor::length(distanceTraveled)) {
+	if (thor::length(mPath.back() - getPosition()) < POINT_REACHED_DISTANCE) {
 		mPath.pop_back();
-			(!mPath.empty())
+		if (mPath.empty())
-				? setSpeed(mPath.back() - getPosition(), mMovementSpeed)
+			setSpeed(sf::Vector2f(), 0);
-				: setSpeed(sf::Vector2f(), 0);
-		}
 	}
 }
 

source/abstract/Character.h

@@ -52,6 +52,9 @@ private:
 	void move();
 
 private:
+	/// Distance to a path point where it will be considered as reached (to
+	/// avoid floating point equality check).
+	static const float POINT_REACHED_DISTANCE;
 	friend class World;
 	World& mWorld;
 	Pathfinder& mPathfinder;

source/abstract/Sprite.cpp

@@ -133,6 +133,10 @@ Sprite::setSpeed(sf::Vector2f direction, float speed) {
 	mSpeed = direction;
 }
 
+/**
+ * Rotates sprite in the direction of the vector. Vector length must not be null,
+ * but is otherwise meaningless.
+ */
 void
 Sprite::setDirection(const sf::Vector2f& direction) {
 	if (direction != sf::Vector2f())

source/sprites/Enemy.cpp

@@ -41,6 +41,8 @@ Enemy::onThink(int elapsed) {
 		else if (!isMoving())
 			setDestination(target->getPosition());
 	}
-	else
+	else {
 		releaseTrigger();
+		setSpeed(sf::Vector2f(), 0);
+	}
 }