# HG changeset patch # User Martin von Zweigbergk # Date 1428471625 25200 # Node ID ff7badaf315838e279e52e0b8f6d87fb2f183c09 # Parent 4eaea0ed8dc10fc796c86e45e132ec2c10b6930c 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(). diff -r 4eaea0ed8dc1 -r ff7badaf3158 mercurial/manifest.py --- 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