Changeset 455

Timestamp:

22-05-11 17:16:00 (2 years ago)

Author:

thomas

Message:

• morituri/program/cdparanoia.py: Since stop can be same as start, add + 1 to progress calculation. Fiexes #37.

Location:

trunk

Files:

• ChangeLog (modified) (1 diff)
• morituri/program/cdparanoia.py (modified) (2 diffs)

trunk/ChangeLog

@@ +1 @@
+2011-05-22  Thomas Vander Stichele  <thomas at apestaart dot org>
+
+        * morituri/program/cdparanoia.py:
+          Since stop can be same as start, add + 1 to progress calculation.
+          Fiexes #37.
+
 2011-05-22  Thomas Vander Stichele  <thomas at apestaart dot org>
 

trunk/morituri/program/cdparanoia.py

@@ -204 +204 @@
         @param start:  first frame to rip
         @type  start:  int
-        @param stop:   last frame to rip (inclusive)
+        @param stop:   last frame to rip (inclusive); >= start
         @type  stop:   int
         @param offset: read offset, in samples
@@ -306 +306 @@
                 self._popen.terminate()
 
-            num = float(self._parser.wrote) - self._start
-            den = float(self._stop) - self._start
-            progress = num / den
+            num = self._parser.wrote - self._start + 1
+            den = self._stop - self._start + 1
+            assert den != 0, "stop %d should be >= start %d" % (
+                self._stop, self._start)
+            progress = float(num) / float(den)
             if progress < 1.0:
                 self.setProgress(progress)