Land on master: FFL_MIN and FFL_MAX are pointless and stupid.
authorTom Sepez <tsepez@chromium.org>
Tue, 7 Apr 2015 23:58:02 +0000 (16:58 -0700)
committerTom Sepez <tsepez@chromium.org>
Tue, 7 Apr 2015 23:58:02 +0000 (16:58 -0700)
Original CL was accidentally based off of XFA.  This CL is
off of master.

TBR=thestig@chromium.org

Review URL: https://codereview.chromium.org/1069553002

fpdfsdk/include/formfiller/FFL_Utils.h
fpdfsdk/src/formfiller/FFL_Utils.cpp

index 4b77aa2..c5ac2a2 100644 (file)
@@ -9,9 +9,6 @@
 
 #define FFL_BASE_USERUNIT                      1.0f / 72.0f
 
-template<class T> T FFL_MIN (const T & i, const T & j) { return ((i < j) ? i : j); }
-template<class T> T FFL_MAX (const T & i, const T & j) { return ((i > j) ? i : j); }
-
 class CFFL_Utils
 {
 public:
index 3ea7891..196102b 100644 (file)
@@ -4,6 +4,8 @@
  
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
+#include <algorithm>
+
 #include "../../include/formfiller/FormFiller.h"
 #include "../../include/formfiller/FFL_Utils.h"
 
@@ -11,10 +13,10 @@ CPDF_Rect CFFL_Utils::MaxRect(const CPDF_Rect & rect1,const CPDF_Rect & rect2)
 {
        CPDF_Rect rcRet;
 
-       rcRet.left = FFL_MIN(rect1.left,rect2.left);
-       rcRet.bottom = FFL_MIN(rect1.bottom,rect2.bottom);
-       rcRet.right = FFL_MAX(rect1.right,rect2.right);
-       rcRet.top = FFL_MAX(rect1.top,rect2.top);
+       rcRet.left = std::min(rect1.left, rect2.left);
+       rcRet.bottom = std::min(rect1.bottom, rect2.bottom);
+       rcRet.right = std::max(rect1.right, rect2.right);
+       rcRet.top = std::max(rect1.top, rect2.top);
 
        return rcRet;
 }
@@ -39,27 +41,6 @@ CPDF_Rect CFFL_Utils::DeflateRect(const CPDF_Rect & crRect,const FX_FLOAT & fSiz
        return crNew;
 }
 
-/*
-FX_BOOL CFFL_Utils::RectContainsRect(const CPDF_Rect & father,const CPDF_Rect & son)
-{
-       return (father.left <= son.left && father.right >= son.right && 
-                               father.bottom <= son.bottom && father.top >= son.top);
-
-}
-
-FX_BOOL CFFL_Utils::RectContainsPoint(const CPDF_Rect & father,const CPDF_Point & son)
-{
-       return (father.left <= son.x && father.right >= son.x && 
-                               father.bottom <= son.y && father.top >= son.y);
-}
-
-FX_BOOL CFFL_Utils::RectContainsXY(const CPDF_Rect & father,FX_FLOAT x,FX_FLOAT y)
-{
-       return (father.left <= x && father.right >= x && 
-                               father.bottom <= y && father.top >= y);
-}
-*/
-
 FX_BOOL CFFL_Utils::TraceObject(CPDF_Object* pObj)
 {
        if (!pObj) return FALSE;