[PATCH 1 of 2] largefiles: don't explicitly list optional parameters that are not used

Augie Fackler raf at durin42.com
Tue Feb 23 14:30:41 EST 2016


On Tue, Feb 23, 2016 at 12:13:59AM -0800, Tony Tung wrote:
> # HG changeset patch
> # User Tony Tung <tonytung at merly.org>
> # Date 1456211899 28800
> #      Mon Feb 22 23:18:19 2016 -0800
> # Node ID cbc605f28b61cc469e21186982d0569bf94f7d2a
> # Parent  169a238c641540d1e78797fbce0f2599e60c85f6
> largefiles: don't explicitly list optional parameters that are not used

Queued this one, waiting for feedback from others on patch 2.

>
> This makes it easier for changes to the API.
>
> diff --git a/hgext/largefiles/overrides.py b/hgext/largefiles/overrides.py
> --- a/hgext/largefiles/overrides.py
> +++ b/hgext/largefiles/overrides.py
> @@ -452,12 +452,12 @@
>  # writing the files into the working copy and lfcommands.updatelfiles
>  # will update the largefiles.
>  def overridecalculateupdates(origfn, repo, p1, p2, pas, branchmerge, force,
> -                             acceptremote, followcopies, matcher=None,
> -                             mergeforce=False):
> +                             acceptremote, followcopies,
> +                             *args, **kwargs):
>      overwrite = force and not branchmerge
>      actions, diverge, renamedelete = origfn(
>          repo, p1, p2, pas, branchmerge, force, acceptremote,
> -        followcopies, matcher=matcher)
> +        followcopies, *args, **kwargs)
>
>      if overwrite:
>          return actions, diverge, renamedelete
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel at mercurial-scm.org
> https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel


More information about the Mercurial-devel mailing list