From: Plom Heller <plom@plomlompom.com>
Date: Mon, 2 Nov 2026 20:34:02 +0000 (+0100)
Subject: Use PageRange.end == None for open end rather than confusing -1.
X-Git-Url: https://plomlompom.com/repos/processes?a=commitdiff_plain;ds=inline;p=bookmaker

Use PageRange.end == None for open end rather than confusing -1.
---

diff --git a/src/bookmaker/misc.py b/src/bookmaker/misc.py
index a291bf7..072d0a8 100755
--- a/src/bookmaker/misc.py
+++ b/src/bookmaker/misc.py
@@ -48,7 +48,7 @@ _TOK_RANGE_END = 'end'
 class PageRange:
     'Section of multi-page document.'
     start = 0
-    end = -1
+    end: Optional[int] = None
 
     def __init__(
             self,
@@ -90,16 +90,16 @@ class PageRange:
             end_max: int
             ) -> None:
         'Define outer limit for page end.'
-        if self.end > end_max:
-            self._fail(f'exceeds maximum {end_max}')
-        elif self.end < 0:
+        if self.end is None:
             self.end = end_max
+        elif self.end > end_max:
+            self._fail(f'exceeds maximum {end_max}')
 
     def ranging(
             self
             ) -> range:
         'Turn into range().'
-        assert self.end >= 0
+        assert self.end is not None
         return range(self.start, self.end)