manifestdict.walk: remove now-redundant check for match.files()
When checking whether we can take the fast path of iterating over
matcher files instead of manifest files, we check whether
match.files() is non-empty. However, now that return early for
match.always(), it can only be empty when there are only
include/exclude patterns, but in that case anypats() will be True, so
it's already covered. This makes manifestdict.walk() more similar to
manifestdict.matches().
--- a/mercurial/manifest.py Tue Apr 07 21:08:23 2015 -0700
+++ b/mercurial/manifest.py Tue Apr 07 22:40:25 2015 -0700
@@ -230,7 +230,7 @@
fset = set(match.files())
# avoid the entire walk if we're only looking for specific files
- if fset and not match.anypats() and util.all(fn in self for fn in fset):
+ if not match.anypats() and util.all(fn in self for fn in fset):
for fn in sorted(fset):
yield fn
return