From bd784f3c2fb002e732f285ac09a8fce2322edfaf Mon Sep 17 00:00:00 2001 From: Sophist Date: Wed, 17 May 2017 21:33:36 +0100 Subject: [PATCH] Use enumerate(x) instead of range(len(x)) --- picard/cluster.py | 4 ++-- picard/ui/tagsfromfilenames.py | 5 ++--- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/picard/cluster.py b/picard/cluster.py index f5705b866..3bd9d40bd 100644 --- a/picard/cluster.py +++ b/picard/cluster.py @@ -248,8 +248,8 @@ class Cluster(QtCore.QObject, Item): # Arrange tracks into albums albums = {} - for i in range(len(tracks)): - cluster = album_cluster_engine.getClusterFromId(tracks[i][1]) + for i, track in enumerate(tracks): + cluster = album_cluster_engine.getClusterFromId(track[1]) if cluster is not None: albums.setdefault(cluster, []).append(i) diff --git a/picard/ui/tagsfromfilenames.py b/picard/ui/tagsfromfilenames.py index 808c7eae8..2ff30f9b6 100644 --- a/picard/ui/tagsfromfilenames.py +++ b/picard/ui/tagsfromfilenames.py @@ -113,9 +113,8 @@ class TagsFromFileNamesDialog(PicardDialog): self.ui.files.setHeaderLabels([_("File Name")] + list(map(display_tag_name, columns))) for item, file in zip(self.items, self.files): matches = self.match_file(file, format) - for i in range(len(columns)): - value = matches.get(columns[i], '') - item.setText(i + 1, value) + for i, column in enumerate(columns): + item.setText(i + 1, matches.get(column, '')) self.ui.files.header().resizeSections(QtWidgets.QHeaderView.ResizeToContents) self.ui.files.header().setStretchLastSection(True)