largefiles: drop a redundant check for patterns when removing without --after
authorMatt Harbison <matt_harbison@yahoo.com>
Wed, 26 Nov 2014 19:28:11 -0500
changeset 23713 5f84695734f1
parent 23712 bfce25d25c96
child 23714 e3a0e7e21f54
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.
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: