revlog: don't use slicing to return parents
authorGregory Szorc <gregory.szorc@gmail.com>
Mon, 25 Dec 2017 16:31:14 -0700
changeset 35521 a0fab647a8f1
parent 35520 711149d8e676
child 35522 ab9d8d298510
revlog: don't use slicing to return parents This is the only place we use a slice on index entries, which are currently tuples. In preparation for moving away from tuples, let's stop using slices so we don't have to implement that support on the new type. We also tweak the logic slightly so the exception only catches the IndexError on the index lookup, not on the index entry lookup. The old code should never have been buggy. But it was semantically wrong. Differential Revision: https://phab.mercurial-scm.org/D1764
mercurial/revlog.py
--- a/mercurial/revlog.py	Thu Jan 04 16:29:03 2018 -0800
+++ b/mercurial/revlog.py	Mon Dec 25 16:31:14 2017 -0700
@@ -622,12 +622,14 @@
 
     def parentrevs(self, rev):
         try:
-            return self.index[rev][5:7]
+            entry = self.index[rev]
         except IndexError:
             if rev == wdirrev:
                 raise error.WdirUnsupported
             raise
 
+        return entry[5], entry[6]
+
     def node(self, rev):
         try:
             return self.index[rev][7]