Mercurial > hg
diff mercurial/statichttprepo.py @ 51900:77a9c7d8a7ba
statichttprepo: stop shadowing the `bytes` builtin
PyCharm flagged it, but I also misunderstood when looking at the code, because
the name implied a byte string, not a number.
author | Matt Harbison <matt_harbison@yahoo.com> |
---|---|
date | Wed, 25 Sep 2024 01:16:47 -0400 |
parents | e26a08563223 |
children | 6641a3ae8313 |
line wrap: on
line diff
--- a/mercurial/statichttprepo.py Wed Sep 25 01:12:39 2024 -0400 +++ b/mercurial/statichttprepo.py Wed Sep 25 01:16:47 2024 -0400 @@ -52,14 +52,14 @@ def seek(self, pos): self.pos = pos - def read(self, bytes: int = -1): + def read(self, n: int = -1): req = urlreq.request(pycompat.strurl(self.url)) end = '' - if bytes == 0: + if n == 0: return b'' - elif bytes > 0: - end = "%d" % (self.pos + bytes - 1) + elif n > 0: + end = "%d" % (self.pos + n - 1) if self.pos or end: req.add_header('Range', 'bytes=%d-%s' % (self.pos, end)) @@ -79,14 +79,14 @@ if code == 200: # HTTPRangeHandler does nothing if remote does not support # Range headers and returns the full entity. Let's slice it. - if bytes > 0 and (self.pos + bytes) < len(data): - data = data[self.pos : self.pos + bytes] + if n > 0 and (self.pos + n) < len(data): + data = data[self.pos : self.pos + n] elif self.pos < len(data): data = data[self.pos :] else: data = b'' - elif 0 < bytes < len(data): - data = data[:bytes] + elif 0 < n < len(data): + data = data[:n] self.pos += len(data) return data