From 4e4f37aa212b32e1414c93a5731b26312df427a6 Mon Sep 17 00:00:00 2001 From: Cosimo Lupo Date: Tue, 18 Jun 2019 16:30:24 +0100 Subject: [PATCH] varLib.merger: use dict comprehension and zip iterator --- Lib/fontTools/varLib/merger.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Lib/fontTools/varLib/merger.py b/Lib/fontTools/varLib/merger.py index c507f5c85..cd16ace45 100644 --- a/Lib/fontTools/varLib/merger.py +++ b/Lib/fontTools/varLib/merger.py @@ -816,10 +816,10 @@ def merge(merger, self, lst): glyphs = singlePosTable.Coverage.glyphs # We know that singlePosTable is Format 2, as this is set # in _Lookup_SinglePos_subtables_flatten. - recs = singlePosTable.Value - numRecs = len(recs) - recList = [ (glyphs[i], recs[i]) for i in range(numRecs)] - singlePosMapping = {gname: valRecord for gname, valRecord in recList} + singlePosMapping = { + gname: valRecord + for gname, valRecord in zip(glyphs, singlePosTable.Value) + } self.SubTable = buildSinglePos(singlePosMapping, merger.font.getReverseGlyphMap()) merger.mergeObjects(self, lst, exclude=['SubTable', 'SubTableCount'])