diff options
| author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2017-06-22 21:38:08 +0000 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2017-06-23 18:27:07 -0700 |
| commit | 55d3426929d4d8c3dec402cabe6fb1bf27d6abad (patch) | |
| tree | 1b037524961e2fc22f891e0e2c2a5dde72f8c0df /builtin/describe.c | |
| parent | 5402b1352f5181247405fbff1887008a0cb3b04a (diff) | |
| download | git-55d3426929d4d8c3dec402cabe6fb1bf27d6abad.tar.gz | |
wildmatch: remove unused wildopts parameterab/wildmatch
Remove the unused wildopts placeholder struct from being passed to all
wildmatch() invocations, or rather remove all the boilerplate NULL
parameters.
This parameter was added back in commit 9b3497cab9 ("wildmatch: rename
constants and update prototype", 2013-01-01) as a placeholder for
future use. Over 4 years later nothing has made use of it, let's just
remove it. It can be added in the future if we find some reason to
start using such a parameter.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/describe.c')
| -rw-r--r-- | builtin/describe.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/builtin/describe.c b/builtin/describe.c index 893c8789f4..2fe05f5324 100644 --- a/builtin/describe.c +++ b/builtin/describe.c @@ -142,7 +142,7 @@ static int get_name(const char *path, const struct object_id *oid, int flag, voi return 0; for_each_string_list_item(item, &exclude_patterns) { - if (!wildmatch(item->string, path + 10, 0, NULL)) + if (!wildmatch(item->string, path + 10, 0)) return 0; } } @@ -158,7 +158,7 @@ static int get_name(const char *path, const struct object_id *oid, int flag, voi return 0; for_each_string_list_item(item, &patterns) { - if (!wildmatch(item->string, path + 10, 0, NULL)) + if (!wildmatch(item->string, path + 10, 0)) break; /* If we get here, no pattern matched. */ |