# HG changeset patch # User Pierre-Yves David # Date 1600932727 -7200 # Node ID 3d5b2b8e93fde28a847064476b93707cadd6c2a2 # Parent c35db907363d1764587360e0bbbb720e48d9674a changing-files: move default constructor value to None According to Augie Fackler, using tuple as default value confuse PyType when actual values are passed at init time. Differential Revision: https://phab.mercurial-scm.org/D9086 diff -r c35db907363d -r 3d5b2b8e93fd mercurial/metadata.py --- a/mercurial/metadata.py Mon Sep 28 11:16:12 2020 +0200 +++ b/mercurial/metadata.py Thu Sep 24 09:32:07 2020 +0200 @@ -40,15 +40,20 @@ """ def __init__( - self, touched=(), added=(), removed=(), p1_copies=(), p2_copies=(), + self, + touched=None, + added=None, + removed=None, + p1_copies=None, + p2_copies=None, ): - self._added = set(added) - self._removed = set(removed) - self._touched = set(touched) + self._added = set(() if added is None else added) + self._removed = set(() if removed is None else removed) + self._touched = set(() if touched is None else touched) self._touched.update(self._added) self._touched.update(self._removed) - self._p1_copies = dict(p1_copies) - self._p2_copies = dict(p2_copies) + self._p1_copies = dict(() if p1_copies is None else p1_copies) + self._p2_copies = dict(() if p2_copies is None else p2_copies) def __eq__(self, other): return (