From 775bdde52ad5375773c0635e4ce52f74cb820525 Mon Sep 17 00:00:00 2001 From: Helmut Grohne Date: Wed, 29 Dec 2021 13:43:48 +0100 Subject: DecompressedStream: avoid mixing types for variable data The local variable data can be bool or bytes. That's inconvenient for static type checkers. Avoid doing so. --- dedup/compression.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/dedup/compression.py b/dedup/compression.py index ea921c4..9cd63e5 100644 --- a/dedup/compression.py +++ b/dedup/compression.py @@ -106,15 +106,13 @@ class DecompressedStream: def _fill_buff_until(self, predicate): assert self.fileobj is not None - data = True - while True: - if predicate(self.buff) or not data: - return + while not predicate(self.buff): data = self.fileobj.read(self.blocksize) if data: self.buff += self.decompressor.decompress(data) elif hasattr(self.decompressor, "flush"): self.buff += self.decompressor.flush() + break def _read_from_buff(self, length): ret = self.buff[:length] -- cgit v1.2.3