swapped two sections for clarity

2019-01-09 14:59:00 +01:00 · 2019-01-09 14:59:00 +01:00 · 55ea94efbb
commit 55ea94efbb
parent 391f7a76c0
1 changed files with 2 additions and 2 deletions
--- a/Lib/fontTools/fontBuilder.py
+++ b/Lib/fontTools/fontBuilder.py
@ -402,11 +402,11 @@ class FontBuilder(object):
        if uvs is not None:
            uvsDict = {}
            for unicodeValue, variationSelector, glyphName in uvs:
-                if variationSelector not in uvsDict:
-                    uvsDict[variationSelector] = []
                if cmapping.get(unicodeValue) == glyphName:
                    # this is a default variation
                    glyphName = None
+                if variationSelector not in uvsDict:
+                    uvsDict[variationSelector] = []
                uvsDict[variationSelector].append((unicodeValue, glyphName))
            uvsSubTable = buildCmapSubTable({}, 14, 0, 5)
            uvsSubTable.uvsDict = uvsDict