[PATCH 2 of 7] context.status: remove unused arguments from _matchstatus()

Martin von Zweigbergk martinvonz at google.com
Sun Nov 2 16:13:46 CST 2014


# HG changeset patch
# User Martin von Zweigbergk <martinvonz at gmail.com>
# Date 1413094733 25200
#      Sat Oct 11 23:18:53 2014 -0700
# Branch stable
# Node ID 3776f277e512ecdc5b2da73f3f3264af3e173640
# Parent  d68ec9fdce58cb01ec349e2731cf09247b10f436
context.status: remove unused arguments from _matchstatus()

diff --git a/mercurial/context.py b/mercurial/context.py
--- a/mercurial/context.py
+++ b/mercurial/context.py
@@ -85,8 +85,7 @@
                 del mf[fn]
         return mf
 
-    def _matchstatus(self, other, s, match, listignored, listclean,
-                     listunknown):
+    def _matchstatus(self, other, match):
         """return match.always if match is none
 
         This internal method provides a way for child objects to override the
@@ -311,9 +310,8 @@
             reversed = True
             ctx1, ctx2 = ctx2, ctx1
 
+        match = ctx2._matchstatus(ctx1, match)
         r = [[], [], [], [], [], [], []]
-        match = ctx2._matchstatus(ctx1, r, match, listignored, listclean,
-                                  listunknown)
         r = ctx2._prestatus(ctx1, r, match, listignored, listclean, listunknown)
         r = ctx2._buildstatus(ctx1, r, match, listignored, listclean,
                               listunknown)
@@ -1472,8 +1470,7 @@
                                                      listunknown)
         return s
 
-    def _matchstatus(self, other, s, match, listignored, listclean,
-                     listunknown):
+    def _matchstatus(self, other, match):
         """override the match method with a filter for directory patterns
 
         We use inheritance to customize the match.bad method only in cases of
@@ -1484,8 +1481,7 @@
         just use the default match object sent to us.
         """
         superself = super(workingctx, self)
-        match = superself._matchstatus(other, s, match, listignored, listclean,
-                                       listunknown)
+        match = superself._matchstatus(other, match)
         if other != self._repo['.']:
             def bad(f, msg):
                 # 'f' may be a directory pattern from 'match.files()',


More information about the Mercurial-devel mailing list