largefiles: avoid using 'lfiles' variable for two purposes
authorMartin von Zweigbergk <martinvonz@gmail.com>
Tue, 16 Sep 2014 22:50:38 -0700
changeset 22524 8c8fe120670f
parent 22523 8cd91f481ffd
child 22525 764127f58903
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.
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):