Use qMax macro instead of reimplementing the same thing

timlinux · timlinux · commit 07b94b2b098d · 2008-02-12T18:12:26.000Z
git-svn-id: http://svn.osgeo.org/qgis/trunk/qgis@8142 c8812cc2-4d05-0410-92ff-de0c093fc19c
diff --git a/src/core/qgspoint.cpp b/src/core/qgspoint.cpp
@@ -99,7 +99,7 @@ int QgsPoint::onSegment(const QgsPoint& a, const QgsPoint& b) const
   //algorithm from 'graphics GEMS', A. Paeth: 'A Fast 2D Point-on-line test'
   if(
      fabs( (b.y() - a.y()) * (m_x - a.x()) - (m_y - a.y()) * (b.x() - a.x())) \
-     >= maxValue( fabs(b.x() - a.x()), fabs(b.y() - a.y()))
+     >= qMax( fabs(b.x() - a.x()), fabs(b.y() - a.y()))
      )
     {
       return 0;
@@ -123,15 +123,3 @@ int QgsPoint::onSegment(const QgsPoint& a, const QgsPoint& b) const
 
   return 2;
 }
-
-double QgsPoint::maxValue(double a, double b) const
-{
-  if(b > a)
-    {
-      return b;
-    }
-  else
-    {
-      return a;
-    }
-}
diff --git a/src/core/qgspoint.h b/src/core/qgspoint.h
@@ -127,9 +127,6 @@ class CORE_EXPORT QgsPoint
     //! y coordinate 
     double m_y;
 
-    //! little helper function that returns the maximum of 
-    //! two doubles (or a in case of equality)
-    double maxValue(double a, double b) const;
     
 }; // class QgsPOint
 
