changeset 38959:0d032756e9bf

dirstate: add comment on why we don't need to check if something is a dir/file Differential Revision: https://phab.mercurial-scm.org/D4161
author Kyle Lippincott <spectral@google.com>
date Wed, 08 Aug 2018 17:07:27 -0700
parents b9f94d67ea73
children d99468d2b09a
files mercurial/dirstate.py
diffstat 1 files changed, 5 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/mercurial/dirstate.py	Wed Aug 08 17:03:05 2018 -0700
+++ b/mercurial/dirstate.py	Wed Aug 08 17:07:27 2018 -0700
@@ -912,6 +912,11 @@
                         continue
                     raise
                 for f, kind, st in entries:
+                    # If we needed to inspect any files, visitentries would have
+                    # been 'this' or 'all', and we would have set it to None
+                    # above. If we have visitentries populated here, we don't
+                    # care about any files in this directory, so no need to
+                    # check the type of `f`.
                     if visitentries and f not in visitentries:
                         continue
                     if normalizefile: