only call scalar finite when necessary

src/pathops/SkPathOpsTypes.h

Index: src/pathops/SkPathOpsTypes.h
diff --git a/src/pathops/SkPathOpsTypes.h b/src/pathops/SkPathOpsTypes.h
index f85c6653eb05d5eae6807b0373abf704ab880c9a..b35c97ef06775adad39e7f6f4da02e9df7ed3047 100644
--- a/src/pathops/SkPathOpsTypes.h
+++ b/src/pathops/SkPathOpsTypes.h
@@ -167,6 +167,11 @@ inline bool AlmostEqualUlps(double a, double b) {
     return AlmostEqualUlps(SkDoubleToScalar(a), SkDoubleToScalar(b));
 }
 
+bool AlmostEqualUlps_Pin(float a, float b);

[reed1, 2015/12/15 14:17:21]

Pin didn't convey anything obvious to me when I saw this and its call-sites.

Not sure I have a better word (other than the ugly _CheckForFinite), so perhaps
we just need some Dox here in the header.

+inline bool AlmostEqualUlps_Pin(double a, double b) {
+    return AlmostEqualUlps_Pin(SkDoubleToScalar(a), SkDoubleToScalar(b));
+}
+
 // Use Almost Dequal when comparing should not special case denormalized values.
 bool AlmostDequalUlps(float a, float b);
 bool AlmostDequalUlps(double a, double b);
@@ -176,6 +181,11 @@ inline bool NotAlmostEqualUlps(double a, double b) {
     return NotAlmostEqualUlps(SkDoubleToScalar(a), SkDoubleToScalar(b));
 }
 
+bool NotAlmostEqualUlps_Pin(float a, float b);
+inline bool NotAlmostEqualUlps_Pin(double a, double b) {
+    return NotAlmostEqualUlps_Pin(SkDoubleToScalar(a), SkDoubleToScalar(b));
+}
+
 bool NotAlmostDequalUlps(float a, float b);
 inline bool NotAlmostDequalUlps(double a, double b) {
     return NotAlmostDequalUlps(SkDoubleToScalar(a), SkDoubleToScalar(b));