<div dir="ltr">This series looks good to me, but I'll let Matt take a look at particularly 1/8, so I'll leave the queuing for him.<br></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jun 4, 2015 at 9:09 PM Matt Harbison <<a href="mailto:mharbison72@gmail.com">mharbison72@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"># HG changeset patch<br>
# User Matt Harbison <<a href="mailto:matt_harbison@yahoo.com" target="_blank">matt_harbison@yahoo.com</a>><br>
# Date 1433470278 14400<br>
#      Thu Jun 04 22:11:18 2015 -0400<br>
# Node ID 89c9da9f2121dcba36002cf10a23df677026ee59<br>
# Parent  91db1c0b272e90c500f1fe5416150a65c2c6579f<br>
largefiles: replace match.bad() monkey patching with match.badmatch()<br>
<br>
No known issues with the previous code since it restored the original method,<br>
but this is cleaner.<br>
<br>
The monkey patching in cat is harmless, because it is created locally, and<br>
doesn't pass it anywhere (subrepo cat isn't supported with largefiles).<br>
<br>
diff --git a/hgext/largefiles/overrides.py b/hgext/largefiles/overrides.py<br>
--- a/hgext/largefiles/overrides.py<br>
+++ b/hgext/largefiles/overrides.py<br>
@@ -100,10 +100,10 @@<br>
             lfmatcher = match_.match(repo.root, '', list(lfpats))<br>
<br>
     lfnames = []<br>
-    m = copy.copy(matcher)<br>
-    m.bad = lambda x, y: None<br>
+    m = matcher<br>
+<br>
     wctx = repo[None]<br>
-    for f in repo.walk(m):<br>
+    for f in repo.walk(match_.badmatch(m, lambda x, y: None)):<br>
         exact = m.exact(f)<br>
         lfile = lfutil.standin(f) in wctx<br>
         nfile = f in wctx<br>
_______________________________________________<br>
Mercurial-devel mailing list<br>
<a href="mailto:Mercurial-devel@selenic.com" target="_blank">Mercurial-devel@selenic.com</a><br>
<a href="https://selenic.com/mailman/listinfo/mercurial-devel" target="_blank">https://selenic.com/mailman/listinfo/mercurial-devel</a><br>
</blockquote></div>