benya/whipper-gui
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #328 from whipper-team/feature/issue-202-zerodivisionerror

Browse Source 
Raise exception when cdparanoia can't read any frames
This commit is contained in:
JoeLametta
2018-11-09 21:52:40 +01:00
committed by GitHub
parent 8cec4696d5 c0637b1b92
commit f1aaaec73a
1 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
whipper/program/cdparanoia.py
View File

@@ -192,16 +192,17 @@ class ProgressParser:
reads = self.reads reads = self.reads
logger.debug('getTrackQuality: frames %d, reads %d' % (frames, reads)) logger.debug('getTrackQuality: frames %d, reads %d' % (frames, reads))
# don't go over a 100%; we know cdparanoia reads each frame at least
# twice
try: try:
# don't go over a 100%
# we know that cdparanoia reads each frame at least twice
return min(frames * 2.0 / reads, 1.0) return min(frames * 2.0 / reads, 1.0)
except ZeroDivisionError: except ZeroDivisionError:
return 0 raise RuntimeError("cdparanoia couldn't read any frames "
"for the current track")
# FIXME: handle errors # FIXME: handle errors
class ReadTrackTask(task.Task): class ReadTrackTask(task.Task):
""" """
I am a task that reads a track using cdparanoia. I am a task that reads a track using cdparanoia.