From d20dec29db84077688deb4f3b0c35cff3cc4f18e Mon Sep 17 00:00:00 2001 From: Cosimo Lupo Date: Thu, 1 Oct 2015 09:29:04 +0100 Subject: [PATCH] [trak] remove redundant if-else block 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. --- Lib/fontTools/ttLib/tables/_t_r_a_k.py | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/Lib/fontTools/ttLib/tables/_t_r_a_k.py b/Lib/fontTools/ttLib/tables/_t_r_a_k.py index 6f1ac5a4d..2e80ff898 100644 --- a/Lib/fontTools/ttLib/tables/_t_r_a_k.py +++ b/Lib/fontTools/ttLib/tables/_t_r_a_k.py @@ -197,16 +197,12 @@ class TrackData(MutableMapping): def toXML(self, writer, ttFont, progress=None): nTracks = len(self) nSizes = len(self.sizes()) - if not any([nTracks, nSizes]): - writer.comment("nTracks=0, nSizes=0") - writer.newline() - else: - writer.comment("nTracks=%d, nSizes=%d" % (nTracks, nSizes)) - writer.newline() - for track, entry in sorted(self.items()): - assert entry.nameIndex is not None - entry.track = track - entry.toXML(writer, ttFont) + writer.comment("nTracks=%d, nSizes=%d" % (nTracks, nSizes)) + writer.newline() + 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): if name != 'trackEntry':