[PATCH 1 of 4] largefiles: use repo[None].walk instead of repo.walk

Martin von Zweigbergk martinvonz at google.com
Sun May 21 01:30:35 EDT 2017


On Fri, May 19, 2017 at 2:39 PM, Augie Fackler <raf at durin42.com> wrote:
> # HG changeset patch
> # User Augie Fackler <augie at google.com>
> # Date 1495144838 14400
> #      Thu May 18 18:00:38 2017 -0400
> # Node ID ea89086c4346e5f38bb1ccb4ea5fa0239ab0c931
> # Parent  531e6a57abd252bef59a5921e3761f1e5d80abba
> largefiles: use repo[None].walk instead of repo.walk
>
> 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 @@ def addlargefiles(ui, repo, isaddremove,
>      m = matcher
>
>      wctx = repo[None]
> -    for f in repo.walk(matchmod.badmatch(m, lambda x, y: None)):
> +    for f in repo[None].walk(matchmod.badmatch(m, lambda x, y: None)):

Nit: could reuse the existing wctx here and in the next patch. I'll
fix that in a followup since I'm still not sure how our tooling keeps
track of accepts across obsmarkers and/or if pushing still menas
accepting everything pushed.

>          exact = m.exact(f)
>          lfile = lfutil.standin(f) in wctx
>          nfile = f in wctx
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel at mercurial-scm.org
> https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel


More information about the Mercurial-devel mailing list