Backup_Repos/beets
1
0
Fork 0
mirror of https://github.com/beetbox/beets.git synced 2025-12-07 00:53:08 +01:00
Code Issues Projects Releases Wiki Activity

artresizer: add FIXME notes where we should probably add some warnings

Browse source 
There's a lot of places where ArtResizer fails or falls back to
completely silently. We should at least log this, or maybe even raise
exceptions in some cases so that the caller can take a decision on how
to handle the failure.
This commit is contained in:
wisp3rwind 2022-03-12 20:59:25 +01:00
parent bc364bc7d4
commit f751893be2
1 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
beets/util/artresizer.py
View file

@ -207,7 +207,7 @@ class IMBackend(LocalBackend):
util.command_output(cmd)
return path_out
except subprocess.CalledProcessError:
# FIXME: add a warning
# FIXME: Should probably issue a warning?
return path_in
def get_format(self, filepath):
@ -219,6 +219,7 @@ class IMBackend(LocalBackend):
try:
return util.command_output(cmd).stdout
except subprocess.CalledProcessError:
# FIXME: Should probably issue a warning?
return None
def convert_format(self, source, target, deinterlaced):
@ -236,6 +237,7 @@ class IMBackend(LocalBackend):
)
return target
except subprocess.CalledProcessError:
# FIXME: Should probably issue a warning?
return source
@property
@ -404,6 +406,7 @@ class PILBackend(LocalBackend):
im.save(py3_path(path_out), progressive=False)
return path_out
except IOError:
# FIXME: Should probably issue a warning?
return path_in
def get_format(self, filepath):
@ -574,6 +577,9 @@ class ArtResizer(metaclass=Shareable):
result_path = self.local_method.convert_format(
path_in, path_new, deinterlaced
)
except Exception:
# FIXME: Should probably issue a warning?
pass
finally:
if result_path != path_in:
os.unlink(path_in)