changeset 11669:c47cb3193c53

localrepo.status: move fixup concatenation inside if block for clarity clean += fixup only makes sense if fixup is not empty; it should be inside the "if fixup" test.
author Nicolas Dumazet <nicdumz.commits@gmail.com>
date Sun, 25 Jul 2010 10:05:38 +0900
parents f070d284994c
children 1b3b843e1100
files mercurial/localrepo.py
diffstat 1 files changed, 3 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/mercurial/localrepo.py	Sat Jul 24 17:23:08 2010 +0200
+++ b/mercurial/localrepo.py	Sun Jul 25 10:05:38 2010 +0900
@@ -1064,11 +1064,11 @@
                     else:
                         fixup.append(f)
 
-                if listclean:
-                    clean += fixup
-
                 # update dirstate for files that are actually clean
                 if fixup:
+                    if listclean:
+                        clean += fixup
+
                     try:
                         # updating the dirstate is optional
                         # so we don't wait on the lock