From 73e11bbf5c031fa6cbcecaed7e83dcd147791fe2 Mon Sep 17 00:00:00 2001 From: Bob Swift Date: Tue, 13 Jul 2021 10:14:28 -0600 Subject: [PATCH] Avoid confusing reuse of variable names in the same method --- picard/ui/options/dialog.py | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/picard/ui/options/dialog.py b/picard/ui/options/dialog.py index 3e5847643..2177e66b5 100644 --- a/picard/ui/options/dialog.py +++ b/picard/ui/options/dialog.py @@ -408,17 +408,17 @@ class AttachedProfilesDialog(PicardDialog): self.setWindowTitle(window_title) for name, title in group_options: - item = QtGui.QStandardItem(_(title)) - item.setEditable(False) - row = [item] + option_item = QtGui.QStandardItem(_(title)) + option_item.setEditable(False) + row = [option_item] attached = [] for profile in profiles: if name in settings[profile["id"]]: attached.append("{0}{1}".format(profile["title"], _(" [Enabled]") if profile["enabled"] else "",)) attached_profiles = "\n".join(attached) if attached else _("None") - item = QtGui.QStandardItem(attached_profiles) - item.setEditable(False) - row.append(item) + profile_item = QtGui.QStandardItem(attached_profiles) + profile_item.setEditable(False) + row.append(profile_item) model.appendRow(row) self.ui.options_list.setModel(model)