[PATCH] match: remove unnecessary setting of self._always

Augie Fackler raf at durin42.com
Tue Oct 7 11:52:51 CDT 2014


On Sat, Oct 04, 2014 at 11:27:32PM -0700, Martin von Zweigbergk wrote:
> # HG changeset patch
> # User Martin von Zweigbergk <martinvonz at gmail.com>
> # Date 1412117888 25200
> #      Tue Sep 30 15:58:08 2014 -0700
> # Node ID cc76b49dd2c20727b35d8d869dd6911845794e13
> # Parent  889789a2ca9f19755bf0d302c8671f850e42a059
> match: remove unnecessary setting of self._always

Sure, looks obviously right. Queued.

>
> The 'always' class calls its parent constructor with an empty list of
> patterns, which will result in a matcher that always matches. The
> parent constructor will set self._always to True in such cases, so
> there is no need to set it again.
>
> diff --git a/mercurial/match.py b/mercurial/match.py
> --- a/mercurial/match.py
> +++ b/mercurial/match.py
> @@ -156,7 +156,6 @@
>  class always(match):
>      def __init__(self, root, cwd):
>          match.__init__(self, root, cwd, [])
> -        self._always = True
>
>  class narrowmatcher(match):
>      """Adapt a matcher to work on a subdirectory only.
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel at selenic.com
> http://selenic.com/mailman/listinfo/mercurial-devel


More information about the Mercurial-devel mailing list