diff options
author | Carlos Martín Nieto <cmn@dwim.me> | 2015-05-20 21:18:25 +0200 |
---|---|---|
committer | Carlos Martín Nieto <cmn@dwim.me> | 2015-05-20 21:49:02 +0200 |
commit | 2c57114f1117ea94e3023d989018748be76d52eb (patch) | |
tree | fcb846a88b016e193a9316af5d3988d4238920a6 | |
parent | cd3f3c2844715b12583eb4d4a6129d3050f115a4 (diff) | |
download | libgit2-cmn/ignored-ignore.tar.gz |
ignore: clear the error when matching a pattern negationcmn/ignored-ignore
When we discover that we want to keep a negative rule, make sure to
clear the error variable, as it we otherwise return whatever was left by
the previous loop iteration.
-rw-r--r-- | src/ignore.c | 1 | ||||
-rw-r--r-- | tests/status/ignore.c | 12 |
2 files changed, 13 insertions, 0 deletions
diff --git a/src/ignore.c b/src/ignore.c index 7ad8500e8..0031e4696 100644 --- a/src/ignore.c +++ b/src/ignore.c @@ -80,6 +80,7 @@ static int does_negate_rule(int *out, git_vector *rules, git_attr_fnmatch *match git_vector_foreach(rules, i, rule) { if (!(rule->flags & GIT_ATTR_FNMATCH_HASWILD)) { if (does_negate_pattern(rule, match)) { + error = 0; *out = 1; goto out; } diff --git a/tests/status/ignore.c b/tests/status/ignore.c index 3193d318e..ba1d69a99 100644 --- a/tests/status/ignore.c +++ b/tests/status/ignore.c @@ -1010,3 +1010,15 @@ void test_status_ignore__subdir_doesnt_match_above(void) cl_git_pass(git_ignore_path_is_ignored(&ignored, g_repo, "src/SRC/test.txt")); cl_assert_equal_i(icase, ignored); } + +void test_status_ignore__negate_exact_previous(void) +{ + int ignored; + + g_repo = cl_git_sandbox_init("empty_standard_repo"); + + cl_git_mkfile("empty_standard_repo/.gitignore", "*.com\ntags\n!tags/\n.buildpath"); + cl_git_mkfile("empty_standard_repo/.buildpath", ""); + cl_git_pass(git_ignore_path_is_ignored(&ignored, g_repo, ".buildpath")); + cl_assert_equal_i(1, ignored); +} |