shadowfacts/fonttools
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[trak] remove redundant if-else block

Browse Source 
If nTracks == 0, nSizes is also == 0, the `else` part is sufficient.
There's a limit case when nTracks can be > 0, and nSizes == 0, but it's
unlikely or useless.
This commit is contained in:
Cosimo Lupo 2015-10-01 09:29:04 +01:00
parent 4f5419fdec
commit d20dec29db
1 changed files with 6 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
Lib/fontTools/ttLib/tables/_t_r_a_k.py
View File

@ -197,16 +197,12 @@ class TrackData(MutableMapping):
def toXML(self, writer, ttFont, progress=None): def toXML(self, writer, ttFont, progress=None):
nTracks = len(self) nTracks = len(self)
nSizes = len(self.sizes()) nSizes = len(self.sizes())
if not any([nTracks, nSizes]): writer.comment("nTracks=%d, nSizes=%d" % (nTracks, nSizes))
writer.comment("nTracks=0, nSizes=0") writer.newline()
writer.newline() for track, entry in sorted(self.items()):
else: assert entry.nameIndex is not None
writer.comment("nTracks=%d, nSizes=%d" % (nTracks, nSizes)) entry.track = track
writer.newline() entry.toXML(writer, ttFont)
for track, entry in sorted(self.items()):
assert entry.nameIndex is not None
entry.track = track
entry.toXML(writer, ttFont)
def fromXML(self, name, attrs, content, ttFont): def fromXML(self, name, attrs, content, ttFont):
if name != 'trackEntry': if name != 'trackEntry':