From 9d152e2773f28d4fb7066010d2ae9099873cb6fd Mon Sep 17 00:00:00 2001
From: Matthias Schiffer <matthias@gamezock.de>
Date: Mon, 4 Jan 2010 01:02:33 +0100
Subject: Corrected edge & vertex collision

---
 src/Collision.cpp |  2 ++
 src/Game.cpp      | 12 ++++++------
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/src/Collision.cpp b/src/Collision.cpp
index ea69b39..e552643 100644
--- a/src/Collision.cpp
+++ b/src/Collision.cpp
@@ -118,6 +118,7 @@ bool Collision::test(const Triangle &t, const vmml::vec3f &m, float r, const vmm
   for(int i = 0; i < 3; ++i) {
     if(testEdge(t.getVertex(i), t.getVertex((i+1)%3), m, r, move, &d)) {
       if(!collision || d < minDistance) {
+        collision = true;
         minDistance = d;
       }
     }
@@ -134,6 +135,7 @@ bool Collision::test(const Triangle &t, const vmml::vec3f &m, float r, const vmm
   for(int i = 0; i < 3; ++i) {
     if(testVertex(t.getVertex(i), m, r, move, &d)) {
       if(!collision || d < minDistance) {
+        collision = true;
         minDistance = d;
       }
     }
diff --git a/src/Game.cpp b/src/Game.cpp
index f583312..aea829f 100644
--- a/src/Game.cpp
+++ b/src/Game.cpp
@@ -131,10 +131,10 @@ void Game::run(int delta) {
 
   vmml::vec3f origMove = playerMove;
 
-  bool ok = false;
+  bool collision = true;
 
-  while(!ok) {
-    ok = true;
+  while(collision) {
+    collision = false;
 
     MathUtil::Plane nearestPlane;
     float nearestDistance;
@@ -150,8 +150,8 @@ void Game::run(int delta) {
           vmml::vec3f intersection = p.intersection(MathUtil::Ray(playerPos, playerMove));
           float distance = intersection.squared_distance(playerPos);
 
-          if(ok || distance < nearestDistance) {
-            ok = false;
+          if(!collision || distance < nearestDistance) {
+            collision = true;
 
             nearestPlane = p;
             nearestDistance = distance;
@@ -164,7 +164,7 @@ void Game::run(int delta) {
       }
     }
 
-    if(!ok) {
+    if(collision) {
       vmml::vec3f move;
 
       if(playerMove.dot(nearestPlane.getNormal()) == 0)
-- 
cgit v1.2.3