Merge pull request #62 from Keeper-of-the-Keys/fix-downloads

Fix bug I introduced when removing asserts (and latent logic error?)

Author: Keeper-of-the-Keys

share/man/man1/gpo.1

@@ -1,4 +1,4 @@
-.TH GPO "1" "September 2024" "gpodder 4.17.1" "User Commands"
+.TH GPO "1" "December 2024" "gpodder 4.17.2" "User Commands"
 .SH NAME
 gpo \- gPodder command-line interface
 .SH SYNOPSIS

src/gpodder/__init__.py

@@ -50,8 +50,8 @@
 # This metadata block gets parsed by setup.py - use single quotes only
 __tagline__ = 'Media and podcast aggregator'
 __author__ = 'Thomas Perl <[email protected]>'
-__version__ = '4.17.1'
-__date__ = '2024-09-10'
+__version__ = '4.17.2'
+__date__ = '2024-12-10'
 __relname__ = 'Achva'
 __copyright__ = '© 2005-2024 Thomas Perl and the gPodder Team'
 __license__ = 'ISC / GPLv3 or later'

src/gpodder/download.py

@@ -107,9 +107,9 @@ class ContentRange(object):
 
     def __init__(self, start, stop, length):
         if start < 0:
-            raise Exception("Bad start: %r" % start)
-        if not all(stop is None or (stop >= 0 and stop >= start)):
-            raise Exception("Bad stop: %r" % stop)
+            raise Exception("Bad start: {}, stop: {}, length: {}".format(str(start), str(stop), str(length)))
+        if stop is None or (stop >= 0 and stop <= start):
+            raise Exception("Bad stop: {}, start: {}, length: {}".format(str(stop), str(start), str(length)))
 
         self.start = start
         self.stop = stop