From e9f077f222513baf07364e91bd528bd5e203535c Mon Sep 17 00:00:00 2001 From: Plom Heller Date: Mon, 2 Nov 2026 21:34:02 +0100 Subject: [PATCH] Use PageRange.end == None for open end rather than confusing -1. --- src/bookmaker/misc.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) 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) -- 2.30.2