ios fixes

src/core/SkPoint.cpp

 
 /*
  * Copyright 2008 The Android Open Source Project
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 
 #include "SkPoint.h"
@@ skipping 150 matching lines @@
 
     float scale;
     if (SkScalarIsFinite(mag2)) {
         scale = length / sk_float_sqrt(mag2);
     } else {
         // our mag2 step overflowed to infinity, so use doubles instead.
         // much slower, but needed when x or y are very large, other wise we
         // divide by inf. and return (0,0) vector.
         double xx = x;
         double yy = y;
+    #ifdef SK_BUILD_FOR_IOS
+        double dscale = length / sqrt(xx * xx + yy * yy);

[reed1, 2014/07/10 18:39:39]

Can we add a comment and/or bug/test ref to explain why we need this?

I wonder if we want a new flag (not necessarily iOS specific) that triggers
this? e.g. SK_FAST_UNDERFLOW_BEHAVIOR? Just a though, which might also help
document what's going on.

[caryclark, 2014/07/10 21:07:03]

Done.

+        fX = x * dscale;
+        fY = y * dscale;
+        return true;
+    #else
         scale = (float)(length / sqrt(xx * xx + yy * yy));
+    #endif
     }
     fX = x * scale;
     fY = y * scale;
     return true;
 }
 
 bool SkPoint::setLengthFast(float length) {
     return this->setLengthFast(fX, fY, length);
 }
 
@@ skipping 66 matching lines @@
 
     if (uDotV <= 0) {
         return v.lengthSqd();
     } else if (uDotV > uLengthSqd) {
         return b.distanceToSqd(*this);
     } else {
         SkScalar det = u.cross(v);
         return SkScalarMulDiv(det, det, uLengthSqd);
     }
 }