Patchwork cleanup: reuse existing wctx variables instead of calling repo[None]

login
register
mail settings
Submitter via Mercurial-devel
Date May 21, 2017, 5:37 a.m.
Message ID <0951b1318886b2797d8d.1495345047@martinvonz.svl.corp.google.com>
Download mbox | patch
Permalink /patch/20780/
State Superseded
Headers show

Comments

via Mercurial-devel - May 21, 2017, 5:37 a.m.
# HG changeset patch
# User Martin von Zweigbergk <martinvonz@google.com>
# Date 1495344472 25200
#      Sat May 20 22:27:52 2017 -0700
# Node ID 0951b1318886b2797d8df4f0dd6c11fb1ea0621a
# Parent  b9942bc6b292aa9b16ad80fceb765dab4f1c777c
cleanup: reuse existing wctx variables instead of calling repo[None]

Patch

diff --git a/hgext/largefiles/overrides.py b/hgext/largefiles/overrides.py
--- a/hgext/largefiles/overrides.py
+++ b/hgext/largefiles/overrides.py
@@ -119,7 +119,7 @@ 
     m = matcher
 
     wctx = repo[None]
-    for f in repo[None].walk(matchmod.badmatch(m, lambda x, y: None)):
+    for f in wctx.walk(matchmod.badmatch(m, lambda x, y: None)):
         exact = m.exact(f)
         lfile = lfutil.standin(f) in wctx
         nfile = f in wctx
diff --git a/mercurial/cmdutil.py b/mercurial/cmdutil.py
--- a/mercurial/cmdutil.py
+++ b/mercurial/cmdutil.py
@@ -715,8 +715,8 @@ 
             badstates = '?'
         else:
             badstates = '?r'
-        m = scmutil.match(repo[None], [pat], opts, globbed=True)
-        for abs in repo[None].walk(m):
+        m = scmutil.match(wctx, [pat], opts, globbed=True)
+        for abs in wctx.walk(m):
             state = repo.dirstate[abs]
             rel = m.rel(abs)
             exact = m.exact(abs)
@@ -3049,7 +3049,7 @@ 
 
         if not m.always():
             matcher = matchmod.badmatch(m, lambda x, y: False)
-            for abs in repo[None].walk(matcher):
+            for abs in wctx.walk(matcher):
                 names[abs] = m.rel(abs), m.exact(abs)
 
             # walk target manifest to fill `names`