Got rid of non-sensical Vector operator overloads

Just because something is the kind of calculation that seems to be
required does not mean it makes sense in general. Let's try to keep
things understandable.

1 files changed, 4 insertions, 2 deletions

diff --git a/src/being.cpp b/src/being.cpp
index 4b972e3d..1c66f673 100644
--- a/src/being.cpp
+++ b/src/being.cpp
@@ -532,13 +532,15 @@ void Being::logic()
         const float nominalLength = dir.length();
 
         // When we've not reached our destination, move to it.
-        if (nominalLength > 1.0f && mWalkSpeed > 0.0f)
+        if (nominalLength > 1.0f && !mWalkSpeed.isNull())
         {
             // The deplacement of a point along a vector is calculated
             // using the Unit Vector (â) multiplied by the point speed.
             // â = a / ||a|| (||a|| is the a length.)
             // Then, diff = (dir/||dir||) * speed.
-            Vector diff = dir.normalized() * mWalkSpeed;
+            const Vector normalizedDir = dir.normalized();
+            Vector diff(normalizedDir.x * mWalkSpeed.x,
+                        normalizedDir.y * mWalkSpeed.y);
 
             // Test if we don't miss the destination by a move too far:
             if (diff.length() > nominalLength)