D5931: branchmap: move __init__ up in branchcache class

pulkit (Pulkit Goyal) phabricator at mercurial-scm.org
Mon Feb 11 13:05:42 UTC 2019


pulkit created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REVISION SUMMARY
  Making __init__ the first function defined helps understanding the class much
  better.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D5931

AFFECTED FILES
  mercurial/branchmap.py

CHANGE DETAILS

diff --git a/mercurial/branchmap.py b/mercurial/branchmap.py
--- a/mercurial/branchmap.py
+++ b/mercurial/branchmap.py
@@ -148,6 +148,21 @@
     This field can be used to avoid changelog reads when determining if a
     branch head closes a branch or not.
     """
+
+    def __init__(self, entries=(), tipnode=nullid, tiprev=nullrev,
+                 filteredhash=None, closednodes=None):
+        super(branchcache, self).__init__(entries)
+        self.tipnode = tipnode
+        self.tiprev = tiprev
+        self.filteredhash = filteredhash
+        # closednodes is a set of nodes that close their branch. If the branch
+        # cache has been updated, it may contain nodes that are no longer
+        # heads.
+        if closednodes is None:
+            self._closednodes = set()
+        else:
+            self._closednodes = closednodes
+
     @classmethod
     def fromfile(cls, repo):
         f = None
@@ -207,20 +222,6 @@
             filename = '%s-%s' % (filename, repo.filtername)
         return filename
 
-    def __init__(self, entries=(), tipnode=nullid, tiprev=nullrev,
-                 filteredhash=None, closednodes=None):
-        super(branchcache, self).__init__(entries)
-        self.tipnode = tipnode
-        self.tiprev = tiprev
-        self.filteredhash = filteredhash
-        # closednodes is a set of nodes that close their branch. If the branch
-        # cache has been updated, it may contain nodes that are no longer
-        # heads.
-        if closednodes is None:
-            self._closednodes = set()
-        else:
-            self._closednodes = closednodes
-
     def validfor(self, repo):
         """Is the cache content valid regarding a repo
 



To: pulkit, #hg-reviewers
Cc: mercurial-devel


More information about the Mercurial-devel mailing list