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.
--- 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