# HG changeset patch # User Matt Harbison # Date 1417048091 18000 # Node ID 5f84695734f155839f8905dc22831e7d5a0746a4 # Parent bfce25d25c96c3fff32cb7844975f6fc7e637c2b largefiles: drop a redundant check for patterns when removing without --after This is in the way of passing a matcher to removelargefiles(). This method is called in exactly two places- first in overrides.addremove() (but only if the pattern list passed to it is not empty), and second in the commands.remove() override. But since the latter calls commands.remove() first, which also does this check, it isn't needed here. diff -r bfce25d25c96 -r 5f84695734f1 hgext/largefiles/overrides.py --- a/hgext/largefiles/overrides.py Wed Dec 31 17:55:43 2014 +0900 +++ b/hgext/largefiles/overrides.py Wed Nov 26 19:28:11 2014 -0500 @@ -156,8 +156,6 @@ def removelargefiles(ui, repo, isaddremove, *pats, **opts): after = opts.get('after') - if not pats and not after: - raise util.Abort(_('no files specified')) m = composelargefilematcher(scmutil.match(repo[None], pats, opts), repo[None].manifest()) try: