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.

1 files 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]