Merge pull request #170 from adobe-type-tools/advance-hashing

Default advance width value to 0 not 1000

Author: khaledhosny

python/psautohint/ufoFont.py

@@ -558,7 +558,7 @@ def get_glyph_bez(glyph, round_coords):
         pen = BezPen(glyph.glyphSet, round_coords)
         glyph.draw(pen)
         if not hasattr(glyph, "width"):
-            glyph.width = 1000
+            glyph.width = 0
         return pen.bez
 
     def _get_or_skip_glyph(self, name, round_coords, doAll):
@@ -803,7 +803,7 @@ class HashPointPen(AbstractPointPen):
 
     def __init__(self, glyph):
         self.glyphset = getattr(glyph, "glyphSet", None)
-        self.width = norm_float(round(getattr(glyph, "width", 1000), 9))
+        self.width = norm_float(round(getattr(glyph, "width", 0), 9))
         self.data = ["w%s" % self.width]
 
     def getHash(self):

tests/integration/data

@@ -304,6 +304,16 @@ def test_hashmap_new_version(tmpdir, caplog):
         hintFiles(options)
 
 
+def test_hashmap_advance(tmpdir):
+    path = "%s/dummy/hashmap_advance.ufo" % DATA_DIR
+    out = str(tmpdir / basename(path)) + ".out"
+    options = Options(path, out)
+
+    hintFiles(options)
+
+    assert differ([path, out])
+
+
 def test_hashmap_transform(tmpdir):
     path = "%s/dummy/hashmap_transform.ufo" % DATA_DIR
     out = str(tmpdir / basename(path)) + ".out"