Glyph positions maintain 32 bit integer part.

include/core/SkScalar.h

Index: include/core/SkScalar.h
diff --git a/include/core/SkScalar.h b/include/core/SkScalar.h
index 3c1787ed2da5ead04788b16ba50cbb8e9116b127..9c51cf9980ae42b86d3d914704a05cc1a210a0d3 100644
--- a/include/core/SkScalar.h
+++ b/include/core/SkScalar.h
@@ -169,11 +169,15 @@ static inline int SkDScalarRoundToInt(SkScalar x) {
 }
 
 static inline SkScalar SkScalarClampMax(SkScalar x, SkScalar max) {
-    return x < 0 ? 0 : x > max ? max : x;
+    x = SkTMin(x, max);
+    x = SkTMax(x, 0);
+    return x;
 }
 
 static inline SkScalar SkScalarPin(SkScalar x, SkScalar min, SkScalar max) {

[bungeman-skia, 2015/03/04 21:38:05]

With clang 3.5 this code turns into branch-less maxss and minss, and the
existing code turns into compare and branch. I couldn't get gcc 4.8.2 to emit
the branch-less version, but this version turns into the same code the old code
did there. Hopefully newer gcc will also optimize this.

[mtklein, 2015/03/04 21:45:30]

Ah, that explains it.  Thanks!  This bit LGTM.

-    return x < min ? min : x > max ? max : x;
+    x = SkTMin(x, max);
+    x = SkTMax(x, min);
+    return x;
 }
 
 SkScalar SkScalarSinCos(SkScalar radians, SkScalar* cosValue);