[PATCH] util: remove unused variable to fix test

Laurent Charignon lcharignon at fb.com
Mon Jan 18 11:06:25 CST 2016


Martin fixed this on Thursday and I didn't pull since.
You can disregard this patch.

Thanks,

Laurent

> On Jan 17, 2016, at 11:17 PM, Laurent Charignon <lcharignon at fb.com> wrote:
> 
> # HG changeset patch
> # User Laurent Charignon <lcharignon at fb.com>
> # Date 1453101121 28800
> #      Sun Jan 17 23:12:01 2016 -0800
> # Node ID 9c46b32a09b2ea357897829ed7cbfe7f4de68877
> # Parent  443848eece189002c542339dc1cf84f49a94c824
> util: remove unused variable to fix test
> 
> Before this patch test-check-pyflakes.t was complaining about a variable being
> unused:
> 
> $ hg locate 'set:**.py or grep("^!#.*python")' 2>/dev/null \
>> | xargs pyflakes 2>/dev/null | "$TESTDIR/filterpyflakes.py"
> +  mercurial/util.py:2696: local variable 'e' is assigned to but never used
> 
> This patch removes the unused variable to make the test pass.
> 
> diff --git a/mercurial/util.py b/mercurial/util.py
> --- a/mercurial/util.py
> +++ b/mercurial/util.py
> @@ -2693,7 +2693,7 @@
>                     exc_type = None
>                     exc_val = None
>                     exc_tb = None
> -            except BaseException as e:
> +            except BaseException:
>                 pending = sys.exc_info()
>                 exc_type, exc_val, exc_tb = pending = sys.exc_info()
>         del self._atexit
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel at selenic.com
> https://selenic.com/mailman/listinfo/mercurial-devel


More information about the Mercurial-devel mailing list