From e1fd96f1ee32e1b8d037b652f79eeaabbc3bcf11 Mon Sep 17 00:00:00 2001 From: Laurent Monin Date: Thu, 7 Sep 2023 10:22:36 +0200 Subject: [PATCH] cover providers: drop debug logging not really needed It makes code complicated, and flood the log output every time options are loaded. --- picard/coverart/providers/__init__.py | 7 ------- 1 file changed, 7 deletions(-) diff --git a/picard/coverart/providers/__init__.py b/picard/coverart/providers/__init__.py index 530b56300..1430f9fba 100644 --- a/picard/coverart/providers/__init__.py +++ b/picard/coverart/providers/__init__.py @@ -29,7 +29,6 @@ from collections import ( namedtuple, ) -from picard import log from picard.config import get_config from picard.coverart.providers.caa import CoverArtProviderCaa from picard.coverart.providers.caa_release_group import ( @@ -81,12 +80,6 @@ def cover_art_providers(): # (yet) unknown providers are placed at the end, disabled ordered_providers = sorted(_cover_art_providers, key=lambda p: order[p.name].position) - def label(p): - checked = 'x' if order[p.name].enabled else ' ' - return "%s [%s]" % (p.name, checked) - - log.debug("CA Providers order: %s", ' > '.join(label(p) for p in ordered_providers)) - for p in ordered_providers: yield ProviderTuple(name=p.name, title=p.title, enabled=order[p.name].enabled, cls=p)