diff options
author | Martin Ågren <martin.agren@gmail.com> | 2020-07-20 20:45:29 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2020-07-20 13:25:07 -0700 |
commit | cada7308ade0b9a018f9c54ea3f226e2e9c39ee5 (patch) | |
tree | 2f8e26c08a19b515d3adbacd3f81a4024e1c0da7 /dir.c | |
parent | completion: fix 'git add' on paths under an untracked directory (diff) | |
download | tgif-cada7308ade0b9a018f9c54ea3f226e2e9c39ee5.tar.xz |
dir: check pathspecs before returning `path_excluded`
In 95c11ecc73 ("Fix error-prone fill_directory() API; make it only
return matches", 2020-04-01), we taught `fill_directory()`, or more
specifically `treat_path()`, to check against any pathspecs so that we
could simplify the callers.
But in doing so, we added a slightly-too-early return for the "excluded"
case. We end up not checking the pathspecs, meaning we return
`path_excluded` when maybe we should return `path_none`. As a result,
`git status --ignored -- pathspec` might show paths that don't actually
match "pathspec".
Move the "excluded" check down to after we've checked any pathspecs.
Reported-by: Andreas Schwab <schwab@linux-m68k.org>
Reviewed-by: Elijah Newren <newren@gmail.com>
Signed-off-by: Martin Ågren <martin.agren@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'dir.c')
-rw-r--r-- | dir.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -2117,13 +2117,13 @@ static enum path_treatment treat_path(struct dir_struct *dir, baselen, excluded, pathspec); case DT_REG: case DT_LNK: - if (excluded) - return path_excluded; if (pathspec && !do_match_pathspec(istate, pathspec, path->buf, path->len, 0 /* prefix */, NULL /* seen */, 0 /* flags */)) return path_none; + if (excluded) + return path_excluded; return path_untracked; } } |