From 11372496e521bb1afcc93ce041b10c646b1e743b Mon Sep 17 00:00:00 2001
From: arrai <array.of.intellect@gmail.com>
Date: Thu, 24 Jun 2010 15:41:12 +0200
Subject: [PATCH] [10104] Cleaned up WorldObject::HasInArc

---
 src/game/MapManager.h    | 15 +++++++++++++++
 src/game/Object.cpp      | 15 ++++++---------
 src/shared/revision_nr.h |  2 +-
 3 files changed, 22 insertions(+), 10 deletions(-)

diff --git a/src/game/MapManager.h b/src/game/MapManager.h
index c48f821ba..a61186ee1 100644
--- a/src/game/MapManager.h
+++ b/src/game/MapManager.h
@@ -112,6 +112,21 @@ class MANGOS_DLL_DECL MapManager : public MaNGOS::Singleton<MapManager, MaNGOS::
             return IsValidMapCoord(loc.mapid,loc.coord_x,loc.coord_y,loc.coord_z,loc.orientation);
         }
 
+        // modulos a radian orientation to the range of 0..2PI
+        static float NormalizeOrientation(float o)
+        {
+            // fmod only supports positive numbers. Thus we have
+            // to emulate negative numbers
+            if(o < 0)
+            {
+                float mod = o *-1;
+                mod = fmod(mod, 2.0f*M_PI_F);
+                mod = -mod+2.0f*M_PI_F;
+                return mod;
+            }
+            return fmod(o, 2.0f*M_PI_F);
+        }
+
         void RemoveAllObjectsInRemoveList();
 
         void LoadTransports();
diff --git a/src/game/Object.cpp b/src/game/Object.cpp
index 5a6492a80..fc7693df4 100644
--- a/src/game/Object.cpp
+++ b/src/game/Object.cpp
@@ -1361,19 +1361,15 @@ bool WorldObject::HasInArc(const float arcangle, const WorldObject* obj) const
     float arc = arcangle;
 
     // move arc to range 0.. 2*pi
-    while( arc >= 2.0f * M_PI_F )
-        arc -=  2.0f * M_PI_F;
-    while( arc < 0 )
-        arc +=  2.0f * M_PI_F;
+    arc = MapManager::NormalizeOrientation(arc);
 
     float angle = GetAngle( obj );
     angle -= m_orientation;
 
     // move angle to range -pi ... +pi
-    while( angle > M_PI_F)
-        angle -= 2.0f * M_PI_F;
-    while(angle < -M_PI_F)
-        angle += 2.0f * M_PI_F;
+    angle = MapManager::NormalizeOrientation(angle);
+    if(angle > M_PI_F)
+        angle -= 2.0f*M_PI_F;
 
     float lborder =  -1 * (arc/2.0f);                       // in range -pi..0
     float rborder = (arc/2.0f);                             // in range 0..pi
@@ -1958,4 +1954,5 @@ bool WorldObject::IsControlledByPlayer() const
         default:
             return false;
     }
-}
\ No newline at end of file
+}
+
diff --git a/src/shared/revision_nr.h b/src/shared/revision_nr.h
index 5e004b61d..9e9c1e875 100644
--- a/src/shared/revision_nr.h
+++ b/src/shared/revision_nr.h
@@ -1,4 +1,4 @@
 #ifndef __REVISION_NR_H__
 #define __REVISION_NR_H__
- #define REVISION_NR "10103"
+ #define REVISION_NR "10104"
 #endif // __REVISION_NR_H__