# HG changeset patch # User Boris Feld # Date 1517848514 -3600 # Node ID 82afb1a5ed94a20f2cf8a4ff05e512654bb55dff # Parent 0f2c51afafb28952f4279de1fe442cb7f437b454 revlog: use context manager for data file lifetime in checksize This is clearer, safer and more modern. diff -r 0f2c51afafb2 -r 82afb1a5ed94 mercurial/revlog.py --- a/mercurial/revlog.py Mon Feb 05 17:34:57 2018 +0100 +++ b/mercurial/revlog.py Mon Feb 05 17:35:14 2018 +0100 @@ -2335,10 +2335,9 @@ expected = max(0, self.end(len(self) - 1)) try: - f = self._datafp() - f.seek(0, 2) - actual = f.tell() - f.close() + with self._datafp() as f: + f.seek(0, 2) + actual = f.tell() dd = actual - expected except IOError as inst: if inst.errno != errno.ENOENT: