path ops -- use standard max, min, double-is-nan

src/pathops/SkPathOpsBounds.h

 /*
  * Copyright 2012 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 #ifndef SkPathOpBounds_DEFINED
 #define SkPathOpBounds_DEFINED
 
 #include "SkPathOpsRect.h"
 #include "SkRect.h"
 
 // SkPathOpsBounds, unlike SkRect, does not consider a line to be empty.
 struct SkPathOpsBounds : public SkRect {
     static bool Intersects(const SkPathOpsBounds& a, const SkPathOpsBounds& b) {
         return a.fLeft <= b.fRight && b.fLeft <= a.fRight &&
                 a.fTop <= b.fBottom && b.fTop <= a.fBottom;
     }
 
-    // FIXME: add() is generically useful and could be added directly to SkRect
+   // Note that add(), unlike SkRect::join() or SkRect::growToInclude()
+   // does not treat the bounds of horizontal and vertical lines as
+   // empty rectangles.
     void add(SkScalar left, SkScalar top, SkScalar right, SkScalar bottom) {
         if (left < fLeft) fLeft = left;
         if (top < fTop) fTop = top;
         if (right > fRight) fRight = right;
         if (bottom > fBottom) fBottom = bottom;
     }
 
     void add(const SkPathOpsBounds& toAdd) {
         add(toAdd.fLeft, toAdd.fTop, toAdd.fRight, toAdd.fBottom);
     }
@@ skipping 21 matching lines @@
         fLeft = fRight = pt.fX;
         fTop = fBottom = pt.fY;
     }
 
     typedef SkRect INHERITED;
 };
 
 extern void (SkPathOpsBounds::*SetCurveBounds[])(const SkPoint[]);
 
 #endif