# HG changeset patch
# User Nicolas Dumazet <nicdumz.commits@gmail.com>
# Date 2010-07-25 01:05:38
# Node ID c47cb3193c534f6899e32e26bf9510da2ef0b7d5
# Parent  f070d284994cbad8a6b65c7a8856dbe199193e03

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.

diff --git a/mercurial/localrepo.py b/mercurial/localrepo.py
--- a/mercurial/localrepo.py
+++ b/mercurial/localrepo.py
@@ -1064,11 +1064,11 @@ class localrepository(repo.repository):
                     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