From eacc3bfa3937caa5f9234642ca12df8d043a06c7 Mon Sep 17 00:00:00 2001
From: mathnerd314 <mathnerd314@837edb03-e0f3-0310-88ca-d4d4e8b29345>
Date: Thu, 4 Mar 2010 02:39:19 +0000
Subject: [PATCH] Small cleanup of Sector::get_nearest_player

git-svn-id: http://supertux.lethargik.org/svn/supertux/trunk/supertux@6537 837edb03-e0f3-0310-88ca-d4d4e8b29345
---
 src/supertux/sector.cpp | 16 ++++++----------
 1 file changed, 6 insertions(+), 10 deletions(-)

diff --git a/src/supertux/sector.cpp b/src/supertux/sector.cpp
index e57058f7e..273c1d45d 100644
--- a/src/supertux/sector.cpp
+++ b/src/supertux/sector.cpp
@@ -1893,13 +1893,11 @@ Sector::get_gravity() const
   return gravity;
 }
 
-Player *
+Player*
 Sector::get_nearest_player (const Vector& pos)
 {
-  Player *nearest_player;
-  float nearest_dist;
-
-  nearest_player = NULL;
+  Player *nearest_player = NULL;
+  float nearest_dist = std::numeric_limits<float>::max();
 
   std::vector<Player*> players = Sector::current()->get_players();
   for (std::vector<Player*>::iterator playerIter = players.begin();
@@ -1907,20 +1905,18 @@ Sector::get_nearest_player (const Vector& pos)
       ++playerIter)
   {
     Player *this_player = *playerIter;
-    float this_dist;
-
     if (this_player->is_dying() || this_player->is_dead())
       continue;
 
-    this_dist = this_player->get_bbox ().distance (pos);
+    float this_dist = this_player->get_bbox ().distance(pos);
 
-    if ((nearest_player == NULL) || (nearest_dist > this_dist)) {
+    if (this_dist < nearest_dist) {
       nearest_player = this_player;
       nearest_dist = this_dist;
     }
   }
 
-  return (nearest_player);
+  return nearest_player;
 } /* Player *get_nearest_player */
 
 std::vector<MovingObject*>
-- 
2.11.0