Patchwork D5541: histedit: drop unused constructor arguments (API)

login
register
mail settings
Submitter phabricator
Date Jan. 10, 2019, 11:26 a.m.
Message ID <e6017ff6bba23d29d2754e66e47f9b63@localhost.localdomain>
Download mbox | patch
Permalink /patch/37614/
State Not Applicable
Headers show

Comments

phabricator - Jan. 10, 2019, 11:26 a.m.
This revision was automatically updated to reflect the committed changes.
Closed by commit rHGa3a24ad10efb: histedit: drop unused constructor arguments (API) (authored by martinvonz, committed by ).

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D5541?vs=13118&id=13130

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

AFFECTED FILES
  hgext/histedit.py

CHANGE DETAILS




To: martinvonz, durin42, #hg-reviewers
Cc: mercurial-devel

Patch

diff --git a/hgext/histedit.py b/hgext/histedit.py
--- a/hgext/histedit.py
+++ b/hgext/histedit.py
@@ -308,21 +308,17 @@ 
     return ''.join(['# %s\n' % l if l else '#\n' for l in lines])
 
 class histeditstate(object):
-    def __init__(self, repo, parentctxnode=None, actions=None, keep=None,
-            topmost=None, replacements=None, lock=None, wlock=None):
+    def __init__(self, repo):
         self.repo = repo
-        self.actions = actions
-        self.keep = keep
-        self.topmost = topmost
-        self.parentctxnode = parentctxnode
-        self.lock = lock
-        self.wlock = wlock
+        self.actions = None
+        self.keep = None
+        self.topmost = None
+        self.parentctxnode = None
+        self.lock = None
+        self.wlock = None
         self.backupfile = None
         self.stateobj = statemod.cmdstate(repo, 'histedit-state')
-        if replacements is None:
-            self.replacements = []
-        else:
-            self.replacements = replacements
+        self.replacements = []
 
     def read(self):
         """Load histedit state from disk and set fields appropriately."""