LGTM - only one optional suggestion http://codereview.appspot.com/6462061/diff/1/java/src/com/google/typography/f... File java/src/com/google/typography/font/tools/fontinfo/FontUtils.java (right): http://codereview.appspot.com/6462061/diff/1/java/src/com/google/typography/f... java/src/com/google/typography/font/tools/fontinfo/FontUtils.java:184: public static double fixed1616ToDouble(int val1616) { This would be ok to move to the sfntly Fixed1616 class.
Moved 16-16 floating point to double functino to Fixed1616 class and uploaded new patch set
LGTM - other than small formatting issue http://codereview.appspot.com/6462061/diff/4001/java/src/com/google/typograph... File java/src/com/google/typography/font/sfntly/math/Fixed1616.java (right): http://codereview.appspot.com/6462061/diff/4001/java/src/com/google/typograph... java/src/com/google/typography/font/sfntly/math/Fixed1616.java:42: return fixed/65536.0; // shift the decimal 16 bits fixed / 65536.0
