Patchwork D901: changelog: use a Factory for default value for files

login
register
mail settings
Submitter phabricator
Date Oct. 4, 2017, 1:16 p.m.
Message ID <3277ecf689a854a8e301abcea5888d02@localhost.localdomain>
Download mbox | patch
Permalink /patch/24459/
State Not Applicable
Headers show

Comments

phabricator - Oct. 4, 2017, 1:16 p.m.
This revision was automatically updated to reflect the committed changes.
Closed by commit rHG50474f0b3f1b: changelog: use a Factory for default value for files (authored by indygreg, committed by ).

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D901?vs=2322&id=2398

REVISION DETAIL
  https://phab.mercurial-scm.org/D901

AFFECTED FILES
  mercurial/changelog.py

CHANGE DETAILS




To: indygreg, #hg-reviewers, lothiraldan
Cc: lothiraldan, mercurial-devel

Patch

diff --git a/mercurial/changelog.py b/mercurial/changelog.py
--- a/mercurial/changelog.py
+++ b/mercurial/changelog.py
@@ -151,7 +151,7 @@ 
     manifest = attr.ib(default=nullid)
     user = attr.ib(default='')
     date = attr.ib(default=(0, 0))
-    files = attr.ib(default=[])
+    files = attr.ib(default=attr.Factory(list))
     description = attr.ib(default='')
 
 class changelogrevision(object):