From patchwork Wed Mar 15 07:01:15 2017
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: [3,of,7] hgk: don't use mutable default argument value
From: Pierre-Yves David <pierre-yves.david@ens-lyon.org>
X-Patchwork-Id: 19348
Message-Id: <b8d26aeff6a637b61cac.1489561275@nodosa.octopoid.net>
To: mercurial-devel@mercurial-scm.org
Date: Wed, 15 Mar 2017 00:01:15 -0700

# HG changeset patch
# User Pierre-Yves David <pierre-yves.david@ens-lyon.org>
# Date 1489560505 25200
#      Tue Mar 14 23:48:25 2017 -0700
# Node ID b8d26aeff6a637b61cac05d5b8710bfc1a120f06
# Parent  910e63ee01566dad50ecb1691d2dc845d88a77be
# EXP-Topic check-pylint
# Available At https://www.mercurial-scm.org/repo/users/marmoute/mercurial/
#              hg pull https://www.mercurial-scm.org/repo/users/marmoute/mercurial/ -r b8d26aeff6a6
hgk: don't use mutable default argument value

Caught by pylint.

diff --git a/hgext/hgk.py b/hgext/hgk.py
--- a/hgext/hgk.py
+++ b/hgext/hgk.py
@@ -71,8 +71,10 @@ testedwith = 'ships-with-hg-core'
     inferrepo=True)
 def difftree(ui, repo, node1=None, node2=None, *files, **opts):
     """diff trees from two commits"""
-    def __difftree(repo, node1, node2, files=[]):
+    def __difftree(repo, node1, node2, files=None):
         assert node2 is not None
+        if files is None:
+            files = []
         mmap = repo[node1].manifest()
         mmap2 = repo[node2].manifest()
         m = scmutil.match(repo[node1], files)