# HG changeset patch # User Martin von Zweigbergk # Date 1410933038 25200 # Node ID 8c8fe120670f320c7f3a775ce54a48610149de65 # Parent 8cd91f481ffd54312513e968c96899cbbdc8472e largefiles: avoid using 'lfiles' variable for two purposes The variable 'lfiles' is first used for a set of the names of all the large files. It is then overwritten with a tuple like the ones returned from status(). To reduce confusion, let's create a separate variable for the second use. diff -r 8cd91f481ffd -r 8c8fe120670f hgext/largefiles/reposetup.py --- a/hgext/largefiles/reposetup.py Mon Sep 22 13:05:36 2014 -0700 +++ b/hgext/largefiles/reposetup.py Tue Sep 16 22:50:38 2014 -0700 @@ -224,9 +224,10 @@ normals = [[fn for fn in filelist if not lfutil.isstandin(fn)] for filelist in result] - lfiles = (modified, added, removed, missing, [], [], clean) + lfstatus = (modified, added, removed, missing, [], [], + clean) result = [sorted(list1 + list2) - for (list1, list2) in zip(normals, lfiles)] + for (list1, list2) in zip(normals, lfstatus)] else: def toname(f): if lfutil.isstandin(f):